Make TexImage2D(ImageBitmap) supports some more formats

Make TexImage2D(ImageBitmap) supports some more formats

With crbug.com/626431 fixed, texImage2D(ImageBitmap) is able to support
more formats for WebGL2.

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

Committed: https://crrev.com/893a850c8714babe8a5d480ce1328cfc8a1e8b92
Cr-Commit-Position: refs/heads/master@{#406858}

[xidachen, 2016-07-21 12:37:40]

Description was changed from

==========
Make TexImage2D(ImageBitmap) supports some more formats

With crbug.com/626431 fixed, texImage2D(ImageBitmap) is able to support
more formats for WebGL2.

BUG=
==========

to

==========
Make TexImage2D(ImageBitmap) supports some more formats

With crbug.com/626431 fixed, texImage2D(ImageBitmap) is able to support
more formats for WebGL2.

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

[xidachen, 2016-07-21 14:01:12]

xidachen@chromium.org changed reviewers:
+ kbr@chromium.org, zmo@chromium.org

[xidachen, 2016-07-21 14:01:13]

PTAL

[Zhenyao Mo, 2016-07-21 14:03:39]

lgtm if tests are passing

https://codereview.chromium.org/2165243002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
(right):

https://codereview.chromium.org/2165243002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp:4527: //
TODO(xidachen): find out why GPU-GPU copy fails on r8_red, rg8_rg, rgb8_rgb,
rgba8_rgba only.
Remove this comment.

[xidachen, 2016-07-21 14:07:40]

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

[commit-bot: I haz the power, 2016-07-21 14:07:55]

Dry run: 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-07-21 15:27:38]

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

[commit-bot: I haz the power, 2016-07-21 15:27:39]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-07-21 15:28:46]

The CQ bit was checked by xidachen@chromium.org

[xidachen, 2016-07-21 15:28:46]

The patchset sent to the CQ was uploaded after l-g-t-m from zmo@chromium.org
Link to the patchset: https://codereview.chromium.org/2165243002/#ps20001
(title: "nits")

[commit-bot: I haz the power, 2016-07-21 15:29:04]

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-07-21 15:32:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-21 15:34:46]

Description was changed from

==========
Make TexImage2D(ImageBitmap) supports some more formats

With crbug.com/626431 fixed, texImage2D(ImageBitmap) is able to support
more formats for WebGL2.

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

to

==========
Make TexImage2D(ImageBitmap) supports some more formats

With crbug.com/626431 fixed, texImage2D(ImageBitmap) is able to support
more formats for WebGL2.

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

Committed: https://crrev.com/893a850c8714babe8a5d480ce1328cfc8a1e8b92
Cr-Commit-Position: refs/heads/master@{#406858}
==========

[commit-bot: I haz the power, 2016-07-21 15:34:47]

Patchset 2 (id:??) landed as
https://crrev.com/893a850c8714babe8a5d480ce1328cfc8a1e8b92
Cr-Commit-Position: refs/heads/master@{#406858}

[Ken Russell (switch to Gerrit), 2016-07-22 00:57:20]

Thanks, LGTM after the fact, but in the future I'd really appreciate it if you
could link to some bug ID. It's hard to backtrack to these CLs that don't have
any associated bug ID when diagnosing issues later.