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

Issue 605222 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Nov 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 605189
issue 663595



Sign in to add a comment

Pause RequestAnimationFrame callback on the engine.

Project Member Reported by khushals...@chromium.org, Apr 20 2016

Issue description

When this is requested Blink queues this callback in the ScriptedAnimationController and schedules it by calling SetNeedsAnimate on the LayerTreeHost which will request a commit. The callback is then run when blink prepares the main frame for the compositor during the BeginMainFrame.
 
Components: Mobile>Blimp>Client
Status: Started (was: Assigned)
Cc: w...@chromium.org vmp...@chromium.org
Looks like this call is on the window and if it is used the client will assume that the engine wants the same frame rate as the client.

Two ways to possibly deal with this, one would be to add a WebSetting to blink so the callback is queued but it does not schedule an animation, https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/core/dom/ScriptedAnimationController.cpp&l=85. If a user gesture triggers an update from the engine the callback will run as usual.

The other would be to simply stop the engine from requesting any commits if it has been put in "paused" state, which would naturally pause these requests as well. Since commit requests are done inside the cc channels, we would need a component which is aware of when we are in this state that the channel on the engine can talk to.
Blocking: 663595
Status: WontFix (was: Started)
Obsolete, WontFix.
Labels: Archive-Blimp

Sign in to add a comment