Bug 1429009. P2 - remove unnecessary AbstractThread::AutoEnter. r=bechen,gerald
authorJW Wang <jwwang@mozilla.com>
Tue, 09 Jan 2018 17:49:53 +0800
changeset 450291 72b59b20a04812e8f0b80bc27df09ebe0f884982
parent 450290 9a6c837be3b44d1fa175b0f00bb405d56ee2960a
child 450292 0b7db28533066166b0a16f787269b5bfbe685869
push id8527
push userCallek@gmail.com
push dateThu, 11 Jan 2018 21:05:50 +0000
treeherdermozilla-beta@95342d212a7a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbechen, gerald
bugs1429009
milestone59.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1429009. P2 - remove unnecessary AbstractThread::AutoEnter. r=bechen,gerald It is not needed since we are already in the right context. MozReview-Commit-ID: 6QXxLMQHavv
dom/media/ChannelMediaDecoder.cpp
--- a/dom/media/ChannelMediaDecoder.cpp
+++ b/dom/media/ChannelMediaDecoder.cpp
@@ -434,17 +434,16 @@ ChannelMediaDecoder::DownloadProgressed(
       __func__,
       [ =, self = RefPtr<ChannelMediaDecoder>(this) ](MediaStatistics aStats) {
         if (IsShutdown()) {
           return;
         }
         mCanPlayThrough = aStats.CanPlayThrough();
         GetStateMachine()->DispatchCanPlayThrough(mCanPlayThrough);
         mResource->ThrottleReadahead(ShouldThrottleDownload(aStats));
-        AbstractThread::AutoEnter context(AbstractMainThread());
         // Update readyState since mCanPlayThrough might have changed.
         GetOwner()->UpdateReadyState();
       },
       []() { MOZ_ASSERT_UNREACHABLE("Promise not resolved"); });
 }
 
 /* static */ ChannelMediaDecoder::PlaybackRateInfo
 ChannelMediaDecoder::ComputePlaybackRate(const MediaChannelStatistics& aStats,