Add an heuristic for promoting canvases to GPU acceleration

Add an heuristic for promoting canvases to GPU acceleration

This change adds a new condition under which 2D canvases should be
promoted to GPU acceleration: if a gpu-accelerated source is drawn to
it. There was a pre-existing heuristic for disabling acceleration on
the source if the source is a 2D canvas. Promoting the destination,
if possible, is usually preferable. This also has benefits when the
source is a WebGL canvas or a video.

BUG=464835, 524628
Review-Url: https://codereview.chromium.org/2625873005
Cr-Commit-Position: refs/heads/master@{#443923}
Committed: https://chromium.googlesource.com/chromium/src/+/41d279a5476937a3981a8413be722d42da0de0d2

[Justin Novosad, 2017-01-11 20:43:13]

junov@chromium.org changed reviewers:
+ xlai@chromium.org

[Justin Novosad, 2017-01-11 20:43:14]

PTAL.

Note: CL to land after the related benchmark has landed and simmered on the bots
for a while: https://codereview.chromium.org/2625093003/

[Justin Novosad, 2017-01-11 21:03:25]

Description was changed from

==========
Add an heuristic for promoting canvases to GPU acceleration

This change adds a new condition under which 2D canvases should be
promoted to GPU acceleration: if a gpu-accelerated source is drawn to
it. There was a pre-existing heuristic for disabling acceleration on
the source if the source is a 2D canvas. Promoting the destination,
if possible, is usually preferable. This also has benefits when the
source is a WebGL canvas or a video.

BUG=464835
==========

to

==========
Add an heuristic for promoting canvases to GPU acceleration

This change adds a new condition under which 2D canvases should be
promoted to GPU acceleration: if a gpu-accelerated source is drawn to
it. There was a pre-existing heuristic for disabling acceleration on
the source if the source is a 2D canvas. Promoting the destination,
if possible, is usually preferable. This also has benefits when the
source is a WebGL canvas or a video.

BUG=464835, 524628
==========

[xlai (Olivia), 2017-01-12 17:50:08]

lgtm.

[Justin Novosad, 2017-01-13 16:12:00]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2017-01-13 16:12:36]

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

[commit-bot: I haz the power, 2017-01-13 17:26:10]

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

[commit-bot: I haz the power, 2017-01-13 17:26:11]

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-01-13 18:42:06]

The CQ bit was checked by junov@chromium.org

[Justin Novosad, 2017-01-13 18:42:06]

The patchset sent to the CQ was uploaded after l-g-t-m from xlai@chromium.org
Link to the patchset: https://codereview.chromium.org/2625873005/#ps20001
(title: "test fix + formatting")

[commit-bot: I haz the power, 2017-01-13 18:42:44]

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

[commit-bot: I haz the power, 2017-01-13 20:18:17]

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

[commit-bot: I haz the power, 2017-01-13 20:18:18]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Justin Novosad, 2017-01-16 16:45:41]

The CQ bit was checked by junov@chromium.org

[Justin Novosad, 2017-01-16 16:45:41]

The patchset sent to the CQ was uploaded after l-g-t-m from xlai@chromium.org
Link to the patchset: https://codereview.chromium.org/2625873005/#ps40001
(title: "webgl test fix")

[commit-bot: I haz the power, 2017-01-16 16:45:58]

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

[commit-bot: I haz the power, 2017-01-16 18:13:55]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484585141687930,
"parent_rev": "847b8a76e4db2b92fef928dedd6afd2bd935add2", "commit_rev":
"41d279a5476937a3981a8413be722d42da0de0d2"}

[commit-bot: I haz the power, 2017-01-16 18:14:35]

Description was changed from

==========
Add an heuristic for promoting canvases to GPU acceleration

This change adds a new condition under which 2D canvases should be
promoted to GPU acceleration: if a gpu-accelerated source is drawn to
it. There was a pre-existing heuristic for disabling acceleration on
the source if the source is a 2D canvas. Promoting the destination,
if possible, is usually preferable. This also has benefits when the
source is a WebGL canvas or a video.

BUG=464835, 524628
==========

to

==========
Add an heuristic for promoting canvases to GPU acceleration

This change adds a new condition under which 2D canvases should be
promoted to GPU acceleration: if a gpu-accelerated source is drawn to
it. There was a pre-existing heuristic for disabling acceleration on
the source if the source is a 2D canvas. Promoting the destination,
if possible, is usually preferable. This also has benefits when the
source is a WebGL canvas or a video.

BUG=464835, 524628

Review-Url: https://codereview.chromium.org/2625873005
Cr-Commit-Position: refs/heads/master@{#443923}
Committed:
https://chromium.googlesource.com/chromium/src/+/41d279a5476937a3981a8413be72...
==========

[commit-bot: I haz the power, 2017-01-16 18:14:36]

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