Mobile/Firefox for Android/shutdown decoder: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
No edit summary
Line 8: Line 8:
The trade-off is that, while the suspended-video-element is switched back to be visible again, we should resume its original video decoder.  
The trade-off is that, while the suspended-video-element is switched back to be visible again, we should resume its original video decoder.  
The resuming operation must be asynchronous and might be time-consuming which depends on the resolution of the video file and whether it contains audio tracks or not.
The resuming operation must be asynchronous and might be time-consuming which depends on the resolution of the video file and whether it contains audio tracks or not.
== Project stage ==
=== Phase0 ===
In the prototype (Phase 0), we have enabled this feature on the Firefox Nightly channel for any video element. We also add telemetry to collect needed information, especially on the resuming time.
=== Phase1 ===
In Phase1, we are going to enable this feature on the Firefox Release channel for videos that is able to be resumed quickly and the criteria is '''1) videos without audio track''' or '''2) videos with both audio and video tracks but with low resolution (480P for now)'''.
=== Phase2 ===
In the future (Phase 2), our goal is to enable this feature on all videos without observable latency while resuming.


== Working flow ==
== Working flow ==
Line 31: Line 22:


=Target Milestone=
=Target Milestone=
Firefox54 (Phase1)
Firefox55
 
Phase 1: Using a blank video decoder to replace video decoder instead of shutdown decoders directly. In this phase, the mechanism is applied to 1) Low-resolution video (480P) and 2) Video film without audio track
 
Phase 2: We're going to enhance the mechanism and make sure it can apply to all video files


=Engineer Owner=
=Engineer Owner=
Line 43: Line 30:
Ioana Chiorean <ioana.chiorean@softvision.ro><br>
Ioana Chiorean <ioana.chiorean@softvision.ro><br>
Sorina Florean [:sorina] <sorina.florean@softvision.ro>
Sorina Florean [:sorina] <sorina.florean@softvision.ro>


= MVP Scope-Bug Tracking =
= MVP Scope-Bug Tracking =

Revision as of 04:01, 17 March 2017

Overview

Background

Suspending video element's video decoder, when the video element is in background tabs or is invisible even in the foreground tab, is a Firefox feature that reduces CPU/GPU & memory usage.

The mechanism is that, when a video element is invisible, we replace its original video decoder with blank video decoder which only produces white frames with right resolution and right time information. The original video decoder is released and the black video decoder is light so that we reduce CPU/GPU & memory usage.

Trade-off

The trade-off is that, while the suspended-video-element is switched back to be visible again, we should resume its original video decoder. The resuming operation must be asynchronous and might be time-consuming which depends on the resolution of the video file and whether it contains audio tracks or not.

Working flow

The following is a step-by-step description of suspending decoder working flow.

  1. At the very beginning of the decoding framework, the raw media data is sent to demuxer.
  2. Demuxer helps to separate a combined signal, e.g., a streaming can be separated into audio data and video data by demuxer. After that, audio and video data are sent to audio and video decoder separately.
  3. In data decoding period, audio decoder keeps working as normal because a user may be listening to the music. But, Firefox uses a blank video decoder to replace current video decoder if the video element is invisible.

Fx Shutdown Decoder Architecture v2.png

Overall Project Health

[GREEN]

Target Milestone

Firefox55

Engineer Owner

Tzuhao Kuo [:kaku]

QA Contact

Ioana Chiorean <ioana.chiorean@softvision.ro>
Sorina Florean [:sorina] <sorina.florean@softvision.ro>

MVP Scope-Bug Tracking

  • bug 1293963 -[Meta] Suspend-video-decoder: phase-1 shipping
Full Query
ID Priority Summary Status Resolution Assigned to Last change time
1276556 P3 [META] Tracking enable of background tab video decoder suspend NEW 2022-10-11T18:31:13Z
1294349 -- Telemetry to support background video decoder suspend: Recovery time from video-decode-suspended RESOLVED FIXED Gerald Squelart (he/him) (not at Mozilla since 2022-09-15) 2016-08-25T07:57:28Z
1294656 -- Seek to nearest keyframe when resuming videos with no audio - no audio track RESOLVED FIXED Tzuhao Kuo [:kaku] 2016-09-05T03:51:20Z
1305338 P1 [UX] UX Spec for Shutdown Decoder Phase 1 RESOLVED FIXED Mark Liang(:mark_liang) 2017-03-10T06:42:07Z
1309492 P3 If a video element's video is suspended, make it draw black frames. RESOLVED WONTFIX Tzuhao Kuo [:kaku] 2017-03-30T08:23:32Z
1309494 P3 Use spinning loading UI if resuming video element's video decoder is longer than 250ms. RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-05-17T09:08:05Z
1345403 -- Mark a video element as taint and never suspend it again if it is passed into drawImage() RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-30T07:46:19Z
1346116 -- Consider a video element is in tree or not to suspend its video decoder RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-14T13:27:37Z
1346120 -- If a video element's video is suspended, keep the last decoded video frame on the screen RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-30T09:00:23Z
1346235 P1 Fennec has stopped working while playing videos VERIFIED FIXED John Lin [:jhlin][:jolin] 2020-12-21T18:31:00Z
1348432 P1 Missing audio in YouTube videos that were requested within the same tab. Refreshing page fixes audio. VERIFIED FIXED Tzuhao Kuo [:kaku] 2017-04-27T07:31:42Z
1348864 P1 Black frame displayed on video decoder resume RESOLVED DUPLICATE Tzuhao Kuo [:kaku] 2020-12-21T18:31:00Z
1349456 -- Mark a video element as taint and never suspend it again if it is aptured via mozCaptureStream() RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-30T07:46:19Z
1349459 -- Mark a video element as taint and never suspend it again if it is passed into createImageBitmap() VERIFIED FIXED Tzuhao Kuo [:kaku] 2017-04-24T08:10:15Z
1365581 P3 [Shutdown Decoder] OGV video elements are not suspended. RESOLVED INVALID Tzuhao Kuo [:kaku] 2020-12-21T18:31:00Z
1365584 P1 [Shutdown Decoder] On WEBM video element resume the player returns a error RESOLVED DUPLICATE Tzuhao Kuo [:kaku] 2020-12-21T18:31:00Z
1369970 -- [Shutdown Decoder] enable the preference. RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-06-08T14:09:25Z

17 Total; 1 Open (5.88%); 13 Resolved (76.47%); 3 Verified (17.65%);


Full Query
ID Priority Summary Status Resolution Assigned to Last change time
1309492 P3 If a video element's video is suspended, make it draw black frames. RESOLVED WONTFIX Tzuhao Kuo [:kaku] 2017-03-30T08:23:32Z
1309494 P3 Use spinning loading UI if resuming video element's video decoder is longer than 250ms. RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-05-17T09:08:05Z
1344357 P1 Closing a content window with a seeking video leaks the window VERIFIED FIXED Tzuhao Kuo [:kaku] 2017-05-24T23:15:07Z
1345034 -- Label runnables in dom/media/WMFMediaDataDecoder.cpp RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-29T06:28:46Z
1345179 -- Crash in OOM | large | mozalloc_abort | mozalloc_handle_oom | moz_xmalloc | mozilla::MakeUnique<T> RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-31T12:53:03Z
1345339 -- Label runnables in dom/media/VideoUtils.h RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-29T06:32:06Z
1345403 -- Mark a video element as taint and never suspend it again if it is passed into drawImage() RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-30T07:46:19Z
1345768 -- Fix build error in H264Converter.h/cpp under unified builds. RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-10T06:42:07Z
1346116 -- Consider a video element is in tree or not to suspend its video decoder RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-14T13:27:37Z
1346120 -- If a video element's video is suspended, keep the last decoded video frame on the screen RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-30T09:00:23Z
1346498 -- Clean up the suspend-video-decoder call path RESOLVED FIXED Tzuhao Kuo [:kaku] 2017-03-15T17:20:45Z
1346705 P3 Enable suspend-video-decoder related mochitests on autophone NEW 2022-10-11T19:05:31Z

12 Total; 1 Open (8.33%); 10 Resolved (83.33%); 1 Verified (8.33%);


UX Spec https://mozilla.invisionapp.com/share/K48PCVSEM

UX Spec


others

Decoder resuming latency

While the suspended-video-element is switched back to be visible again, we should resume its original video decoder. The resuming operation must be asynchronous since we don't want to block the main thread and might be time-consuming which depends on the resolution of the video file and whether it contains audio tracks or not.

Currently, we have no way to boost the resuming time, however, we have telemetries for collecting the needed time of different resolutions on different platforms.