Bug 1352934 - make the seek operation at StateObject::HandleResumeVideoDecoding() unobservable; r=jwwang

The seeking operation WAS unobservable before Bug 1309494.
It was changed to be observable because I want the "canplay" event, which will be dispatched after the seeking operation is done,
to notify our video controller to cancel the throbber(which will be shown if the decoder resuming cannot be done in 250 ms).

Now, we can change the seeking operatoin back to unobservable thanks to bug 1350852.
In bug 1350852, we dispatche a specific "mozvideoonlyseekcompleted" event to cancel the throbber.

MozReview-Commit-ID: Ke9sqDPotAB

--HG--
extra : rebase_source : a7f4c04b11fa4e2acbb7445152410d94e43e4b16
This commit is contained in:
Kaku Kuo 2017-04-06 15:14:37 +08:00
parent f12b0d6228
commit 90bec4cffc

View File

@ -2042,7 +2042,7 @@ StateObject::HandleResumeVideoDecoding(const TimeUnit& aTarget)
// invalid after the current state object is deleted in SetState();
RefPtr<AbstractThread> mainThread = mMaster->mAbstractMainThread;
SetSeekingState(Move(seekJob), EventVisibility::Observable)->Then(
SetSeekingState(Move(seekJob), EventVisibility::Suppressed)->Then(
mainThread, __func__,
[start, info, hw](){ ReportRecoveryTelemetry(start, info, hw); },
[](){});