Improve metrics for display list canvas fallback reason

Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/774cb99b4af46bdd0954d13d44de66a8c19cd640
Cr-Commit-Position: refs/heads/master@{#437714}

[Justin Novosad, 2016-12-06 16:20:23]

Description was changed from

==========
Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
==========

to

==========
Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Justin Novosad, 2016-12-06 16:20:41]

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

[commit-bot: I haz the power, 2016-12-06 16:20:59]

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

[Justin Novosad, 2016-12-06 16:22:06]

junov@chromium.org changed reviewers:
+ jwd@chromium.org, kbr@chromium.org, xidachen@chromium.org

[Justin Novosad, 2016-12-06 16:22:06]

PTAL

jwd: histograms.xml
kbr: modules/webgl
xidachen: everything else

[commit-bot: I haz the power, 2016-12-06 17:25:04]

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

[commit-bot: I haz the power, 2016-12-06 17:25:05]

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

[Justin Novosad, 2016-12-06 20:07:17]

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

[commit-bot: I haz the power, 2016-12-06 20:08:04]

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

[xidachen, 2016-12-06 20:13:29]

lgtm

[commit-bot: I haz the power, 2016-12-06 21:49:02]

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

[commit-bot: I haz the power, 2016-12-06 21:49:03]

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

[Ken Russell (switch to Gerrit), 2016-12-08 08:13:50]

WebGL parts LGTM. Good cleanup of redundant enums.

[Justin Novosad, 2016-12-08 20:21:23]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-12-08 20:21:52]

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

[Justin Novosad, 2016-12-08 20:22:02]

jwd, ping

[commit-bot: I haz the power, 2016-12-08 20:58:50]

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

[commit-bot: I haz the power, 2016-12-08 20:58:51]

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

[Ken Russell (switch to Gerrit), 2016-12-09 03:56:44]

kbr@chromium.org changed reviewers:
+ isherman@chromium.org

[Ken Russell (switch to Gerrit), 2016-12-09 03:56:45]

+isherman as another tools/metrics OWNER

[Justin Novosad, 2016-12-09 20:04:48]

metrics ping!

[jwd, 2016-12-09 20:04:50]

lgtm

[Justin Novosad, 2016-12-09 20:08:04]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-12-09 20:09:27]

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, 2016-12-10 01:51:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481314084339010, "parent_rev":
"7ba113e10e862d3b50cc1999c4c550a8bbc53324", "commit_rev":
"e5fa996287faec1e1d3295fa178b6ac961931e14"}

[commit-bot: I haz the power, 2016-12-10 01:51:44]

Description was changed from

==========
Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2557783002
==========

[commit-bot: I haz the power, 2016-12-10 01:51:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-12 14:59:21]

Description was changed from

==========
Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2557783002
==========

to

==========
Improve metrics for display list canvas fallback reason

This change removes the CopyToWebGLTexture fallback reason, which
was no longer capturing the right thing, and replaces it with
more granular fallback reasons for the different variants of texImage.
This CL also adds more specific fallback reasons to track cases where
a canvas is copied to the clipboard and createImageBItmap.

BUG=671637
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/774cb99b4af46bdd0954d13d44de66a8c19cd640
Cr-Commit-Position: refs/heads/master@{#437714}
==========

[commit-bot: I haz the power, 2016-12-12 14:59:23]

Patchset 1 (id:??) landed as
https://crrev.com/774cb99b4af46bdd0954d13d44de66a8c19cd640
Cr-Commit-Position: refs/heads/master@{#437714}