New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 728832 link

Starred by 1 user
Status: Fixed
Owner:
yiyix@chromium.org
Closed: Dec 2017
Cc:
samans@chromium.org
█ fsam...@chromium.org
kylec...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

RenderWidgetHostImpl should receive frame tokens from viz

Project Member Reported by samans@chromium.org, Jun 1 2017 
Currently RenderWidgetHostImpl receives a whole CompositorFrame and extracts the frame token out of it. This will not work if renderer's frames bypass the browser and go to gpu directly. RWHI should receive updates about frame tokens from SurfaceManager/FrameSinkManagerHost, possibly by implementing the SurfaceObserver interface.

Comment 1 by kylec...@chromium.org, Jun 2 2017 

SurfaceManager and SurfaceObserver shouldn't be visible to RWHI in this new world. Those will be in the viz process and RWHI will be in the browser process. FrameSinkManagerHost via FrameSinkObserver should be what sends the updates to RWHI I think?

Comment 2 by samans@chromium.org, Jun 2 2017 

Yes, sounds about right. I mentioned SurfaceManager as well because I wasn't sure we want to tolerate the mojo overhead, but frame tokens I believe are not very time critical so it should be fine.

Comment 3 by fsam...@chromium.org, Jun 13 2017

Components: -Internals>MUS

Comment 4 by fsam...@chromium.org, Dec 21 2017

Owner: yiyix@chromium.org
Status: Fixed (was: Assigned)
yiyix@ fixed this. Assigning to her and marking as FIXED.
►

Sign in to add a comment