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

Issue 720324 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 672962



Sign in to add a comment

Surface Synchronization: RenderWidget::Resize should defer commits (or draws) until it gets a LocalSurfaceId

Project Member Reported by fsam...@chromium.org, May 10 2017

Issue description

Writing this down so I don't forget. RenderWidget::Resize might need a synchronous resize to happen in response to window.resizeBy/resizeTo, or device emulation, etc. We would like to always allocate LocalSurfaceIds in the browser for resize. Thus, the renderer should pass a Resize request to the browser (with ResizeParams + request ID or something). The browser sends a response with a LocalSurfaceId to use.

There's a race here: the renderer can resize itself multiple times before the browser ACKs. Maybe if it gets back a stale response, then it drops it, and waits for the next response.
 
Status: Available (was: Untriaged)

Comment 2 by danakj@chromium.org, May 10 2017

Summary: Surface Synchronization: RenderWidget::Resize should defer commits (or draws) until it gets a LocalSurfaceId (was: Surface Synchronization: RenderWidget::Resize should defer commits until it gets a LocalSurfaceId)
Cc: varkha@chromium.org

Comment 4 by fsamuel@google.com, May 23 2017

Owner: samans@chromium.org
Status: Assigned (was: Available)
Assigning to samans@
Blocking: 672962

Comment 6 by danakj@chromium.org, Jun 12 2017

Owner: ----
Status: Available (was: Assigned)

Comment 7 by danakj@chromium.org, Jun 12 2017

Blocking: -601863
Components: -Internals>MUS

Comment 9 by fsamuel@google.com, Sep 6 2017

Owner: fsam...@chromium.org
Status: Assigned (was: Available)
This has been implemented.

Comment 10 by fsamuel@google.com, Sep 13 2017

Status: Fixed (was: Assigned)

Sign in to add a comment