Bug 1309494 part 0 - make the seek operation at StateObject::HandleResumeVideoDecoding() observable; r=jwwang
authorKaku Kuo <kaku@mozilla.com>
Mon, 13 Mar 2017 18:01:42 +0800
changeset 347465 c02436238eeef12d64856029e661d98bfa5389b6
parent 347464 4bcf6f4ded6411df5fa166933f49e049d9ea23e0
child 347466 32f6036fe3fda95dd103a8cc666ed6e1b7d3aa06
push id31496
push usercbook@mozilla.com
push dateTue, 14 Mar 2017 13:21:57 +0000
treeherdermozilla-central@9a26ed658fdc [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjwwang
bugs1309494
milestone55.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 1309494 part 0 - make the seek operation at StateObject::HandleResumeVideoDecoding() observable; r=jwwang So that the we can listen to the 'canplay' event in the videocontrols.xml. MozReview-Commit-ID: 5T7akeC7EJq
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -2016,17 +2016,17 @@ StateObject::HandleResumeVideoDecoding()
   seekJob.mTarget.emplace(mMaster->GetMediaTime(),
                           type,
                           true /* aVideoOnly */);
 
   // Hold mMaster->mAbstractMainThread here because this->mMaster will be
   // invalid after the current state object is deleted in SetState();
   RefPtr<AbstractThread> mainThread = mMaster->mAbstractMainThread;
 
-  SetSeekingState(Move(seekJob), EventVisibility::Suppressed)->Then(
+  SetSeekingState(Move(seekJob), EventVisibility::Observable)->Then(
     mainThread, __func__,
     [start, info, hw](){ ReportRecoveryTelemetry(start, info, hw); },
     [](){});
 }
 
 RefPtr<MediaDecoder::SeekPromise>
 MediaDecoderStateMachine::
 StateObject::SetSeekingState(SeekJob&& aSeekJob, EventVisibility aVisibility)