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

Commit dc3d6784 authored by James Dong's avatar James Dong Committed by Android (Google) Code Review
Browse files

Merge "Revert "Add some logging to track down where ANR happens during MediaPlayer._release().""

parents 6f93135e a3f4d7f0

media/libstagefright/AwesomePlayer.cpp

+0 −4
Original line number Diff line number Diff line
@@ -453,7 +453,6 @@ status_t AwesomePlayer::setDataSource_l(const sp<MediaExtractor> &extractor) { 
}



void AwesomePlayer::reset() {

    LOGI("reset");

    Mutex::Autolock autoLock(mLock);

    reset_l();

}
@@ -467,10 +466,8 @@ void AwesomePlayer::reset_l() { 
                    Playback::STOP, 0);

            mDecryptHandle = NULL;

            mDrmManagerClient = NULL;

            LOGI("DRM manager client stopped");

    }





    if (mFlags & PLAYING) {

        uint32_t params = IMediaPlayerService::kBatteryDataTrackDecoder;

        if ((mAudioSource != NULL) && (mAudioSource != mAudioTrack)) {
@@ -503,7 +500,6 @@ void AwesomePlayer::reset_l() { 
        mPreparedCondition.wait(mLock);

    }



    LOGI("cancel player events");

    cancelPlayerEvents();



    mWVMExtractor.clear();

media/libstagefright/OMXCodec.cpp

+1 −2
Original line number Diff line number Diff line
@@ -3539,7 +3539,7 @@ status_t OMXCodec::start(MetaData *meta) { 
}



status_t OMXCodec::stop() {

    CODEC_LOGI("stop mState=%d", mState);

    CODEC_LOGV("stop mState=%d", mState);



    Mutex::Autolock autoLock(mLock);
@@ -3601,7 +3601,6 @@ status_t OMXCodec::stop() { 
        mLeftOverBuffer = NULL;

    }



    CODEC_LOGI("stopping video source");

    mSource->stop();



    CODEC_LOGI("stopped in state %d", mState);