Throttle frames in OffscreenCanvasFrameDispatcherImpl

Throttle frames in OffscreenCanvasFrameDispatcherImpl

Propagate gpu/compositor backpressure.  This prevents the browser from
hanging when OffscreenCanvas animations cannot render at 60fps because
they are GPU bound.

This change Makes OffscreenCanvas ignore the BeginFrame signal when
backpressure is detected.  It uses the call to
OffscreenCanvasFrameDispatcherImpl::DidReceiveCompositorFrameAck as a
signal to unblock the next animation frame.

BUG=708709
Review-Url: https://codereview.chromium.org/2817603004
Cr-Commit-Position: refs/heads/master@{#464177}
Committed: https://chromium.googlesource.com/chromium/src/+/b4efc8c9a47145a0d44414db4a6942d31bbde2c8

[Justin Novosad, 2017-04-12 17:38:24]

The CQ bit was checked by junov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-12 17:38:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Justin Novosad, 2017-04-12 17:41:24]

The CQ bit was checked by junov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-12 17:41:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Justin Novosad, 2017-04-12 17:42:16]

junov@chromium.org changed reviewers:
+ eseckler@chromium.org, fsamuel@chromium.org

[Justin Novosad, 2017-04-12 17:42:16]

PTAL

[Eric Seckler, 2017-04-12 19:39:18]

lgtm % nit

https://codereview.chromium.org/2817603004/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
(right):

https://codereview.chromium.org/2817603004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp:410:
// TODO(fsamuel): Implement this.
Is there more to do here? If so, let's update this comment to reflect what's
missing, otherwise remove it.

[commit-bot: I haz the power, 2017-04-12 19:42:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-12 19:42:47]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Justin Novosad, 2017-04-12 20:28:15]

The CQ bit was checked by junov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-12 20:29:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Justin Novosad, 2017-04-12 20:30:28]

The CQ bit was unchecked by junov@chromium.org

[Justin Novosad, 2017-04-12 20:30:35]

The CQ bit was checked by junov@chromium.org

[Justin Novosad, 2017-04-12 20:30:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
eseckler@chromium.org
Link to the patchset: https://codereview.chromium.org/2817603004/#ps40001
(title: "Remove obsolete comment")

[commit-bot: I haz the power, 2017-04-12 20:30:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Fady Samuel, 2017-04-12 21:13:16]

lgtm

[commit-bot: I haz the power, 2017-04-12 22:30:08]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1492029035075160,
"parent_rev": "c18633962408600845d8d1b717acfa36e56b09e4", "commit_rev":
"b4efc8c9a47145a0d44414db4a6942d31bbde2c8"}

[commit-bot: I haz the power, 2017-04-12 22:30:59]

Description was changed from

==========
Throttle frames in OffscreenCanvasFrameDispatcherImpl

Propagate gpu/compositor backpressure.  This prevents the browser from
hanging when OffscreenCanvas animations cannot render at 60fps because
they are GPU bound.

This change Makes OffscreenCanvas ignore the BeginFrame signal when
backpressure is detected.  It uses the call to
OffscreenCanvasFrameDispatcherImpl::DidReceiveCompositorFrameAck as a
signal to unblock the next animation frame.

BUG=708709
==========

to

==========
Throttle frames in OffscreenCanvasFrameDispatcherImpl

Propagate gpu/compositor backpressure.  This prevents the browser from
hanging when OffscreenCanvas animations cannot render at 60fps because
they are GPU bound.

This change Makes OffscreenCanvas ignore the BeginFrame signal when
backpressure is detected.  It uses the call to
OffscreenCanvasFrameDispatcherImpl::DidReceiveCompositorFrameAck as a
signal to unblock the next animation frame.

BUG=708709

Review-Url: https://codereview.chromium.org/2817603004
Cr-Commit-Position: refs/heads/master@{#464177}
Committed:
https://chromium.googlesource.com/chromium/src/+/b4efc8c9a47145a0d44414db4a69...
==========

[commit-bot: I haz the power, 2017-04-12 22:31:00]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/b4efc8c9a47145a0d44414db4a69...