Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 34915b1b authored by hkuang's avatar hkuang
Browse files

transcoding: Make TranscodingTestConfig to be nullable. 

Bug: 145628554
Bug: 154734285
Test: Unit test
Change-Id: I93b5f6a34ad02d82f154d3ae07ed0516b1780db2
parent 2ef2b434

media/libmediatranscoding/aidl/android/media/TranscodingRequestParcel.aidl

+2 −2
Original line number Diff line number Diff line
@@ -74,7 +74,7 @@ parcelable TranscodingRequestParcel { 
    boolean isForTesting = false;



    /**

     * Test configuration. This is only valid when isForTesting is set to true.

     * Test configuration. This will be available only when isForTesting is set to true.

     */

    TranscodingTestConfig testConfig;

     @nullable TranscodingTestConfig testConfig;

}

services/mediatranscoding/SimulatedTranscoder.cpp

+2 −2
Original line number Diff line number Diff line
@@ -50,8 +50,8 @@ void SimulatedTranscoder::setCallback(const std::shared_ptr<TranscoderCallbackIn 
void SimulatedTranscoder::start(

        ClientIdType clientId, JobIdType jobId, const TranscodingRequestParcel& request,

        const std::shared_ptr<ITranscodingClientCallback>& /*clientCallback*/) {

    if (request.testConfig.processingTotalTimeMs > 0) {

        mJobProcessingTimeMs = request.testConfig.processingTotalTimeMs;

    if (request.testConfig.has_value() && request.testConfig->processingTotalTimeMs > 0) {

        mJobProcessingTimeMs = request.testConfig->processingTotalTimeMs;

    }

    ALOGV("%s: job {%d}: processingTime: %lld", __FUNCTION__, jobId,

          (long long)mJobProcessingTimeMs);