New issue
Advanced search Search tips

Issue 669276 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Refactor WebMediaPlayerMS towards a pull model

Project Member Reported by emir...@chromium.org, Nov 28 2016

Issue description

Current we have both push and pull models happening between compositor and MediaStream that is the main cause for jank. MediaStreamVideoTrack pushes the video frames from IO thread to MediaStreamVideoRendererSink. In the meantime, compositor tries to pull current frame from WedbMediaPlayerMSCompositor. To bridge the gap, we end up using another thread. This can ideally be skipped.

 
As video track of the remote stream is naturally a push model, but compositing is a pull model, there must be a pool somewhere (now it is the WebMediaPlayerMS).

Compositing pull happens on compositor thread.
We mainly use IO thread pushing the frames along the code path, but from MediaStreamVideoRendererSink to WebMediaPlayerMS, we use render main thread to do the push.

I think what you mean is that we can avoid using renderer main thread, but stick to using IO thread to push the frame into WebMediaPlayerMS.
Cc: dalecur...@chromium.org
Labels: M-57
Tagging with current canary milestone.
Labels: TE-NeedsTriageHelp
Status: Available (was: Unconfirmed)
Project Member

Comment 6 by sheriffbot@chromium.org, Feb 16 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Untriaged)
This one is already fixed long time ago.

Sign in to add a comment