Minor clean up in transferToImageBitmap API

Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.

TBR=kbr@chromium.org, zmo@chromium.org
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/f92c285443e7186ad66ef8a77ae6a73ea55240da
Cr-Commit-Position: refs/heads/master@{#427188}

[xidachen, 2016-10-22 14:02:15]

Description was changed from

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.
==========

to

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[xidachen, 2016-10-22 14:03:58]

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

[xidachen, 2016-10-22 14:03:58]

PTAL

[Justin Novosad, 2016-10-24 17:54:51]

lgtm

[Justin Novosad, 2016-10-24 17:56:13]

On 2016/10/24 17:54:51, Justin Novosad wrote:
> lgtm

Changes in webGL are trivial. you can TBR a webgl owner IMHO

[xidachen, 2016-10-24 18:13:19]

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

[commit-bot: I haz the power, 2016-10-24 18:13:58]

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

[Ken Russell (switch to Gerrit), 2016-10-24 18:14:38]

On 2016/10/24 17:56:13, Justin Novosad wrote:
> On 2016/10/24 17:54:51, Justin Novosad wrote:
> > lgtm
> 
> Changes in webGL are trivial. you can TBR a webgl owner IMHO

Agree, would be fine to TBR= me or zmo. Regardless, LGTM.

[xidachen, 2016-10-24 18:14:44]

Description was changed from

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.

TBR=kbr@chromium.org, zmo@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[xidachen, 2016-10-24 18:14:44]

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

[commit-bot: I haz the power, 2016-10-24 22:06:39]

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

[commit-bot: I haz the power, 2016-10-24 22:06:39]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-10-25 00:29:39]

The CQ bit was checked by xidachen@chromium.org

[commit-bot: I haz the power, 2016-10-25 00:30:18]

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-10-25 00:35:13]

Description was changed from

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.

TBR=kbr@chromium.org, zmo@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.

TBR=kbr@chromium.org, zmo@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-10-25 00:35:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 00:37:07]

Description was changed from

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.

TBR=kbr@chromium.org, zmo@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Minor clean up in transferToImageBitmap API

The transferToImageBitmap is an API in OffscreenCanvas. This API could
throw exception in certain cases, which means that the
OffscreenCanvas::transferToImageBitmap() method has a ExceptionState&
parameter.

In our current implementation, we do all the exception check in the above
method. However, we still pass the ExceptionState& parameter to the
CanvasRenderingContext::transferToImageBitmap() method, and this method
is overriden by OffscreenCanvasRenderingContext2D, WebGLRenderingContext,
and WebGL2RenderingContext. So this method in all these sub-classes also
take a ExceptionState& parameter but do nothing with it. We should not
need to pass this parameter to CanvasRenderingContext::transferToImageBitmap.

TBR=kbr@chromium.org, zmo@chromium.org
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/f92c285443e7186ad66ef8a77ae6a73ea55240da
Cr-Commit-Position: refs/heads/master@{#427188}
==========

[commit-bot: I haz the power, 2016-10-25 00:37:07]

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