cc: Ensure to reapply a shared canvas transform to image hijack canvas.

cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to apply the matrix to the hijack canvas
before attaching the raster canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7cb1a3b0e0aec2fad7b3f59b789cff12efa3dd12
Cr-Commit-Position: refs/heads/master@{#387390}

[vmpstr, 2016-04-13 23:32:05]

Description was changed from

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to clear the matrix on the raster canvas
and reapply it after wrapping it in the image hijack canvas.

R=enne
BUG=601913
==========

to

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to clear the matrix on the raster canvas
and reapply it after wrapping it in the image hijack canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[vmpstr, 2016-04-13 23:34:24]

Please take a look.

[enne (OOO), 2016-04-14 00:27:44]

lgtm

[vmpstr, 2016-04-14 18:19:24]

Description was changed from

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to clear the matrix on the raster canvas
and reapply it after wrapping it in the image hijack canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to apply the matrix to the hijack canvas
before attaching the raster canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[vmpstr, 2016-04-14 18:22:32]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2016-04-14 18:22:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1888613002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1888613002/20001

[commit-bot: I haz the power, 2016-04-14 19:17:52]

Description was changed from

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to apply the matrix to the hijack canvas
before attaching the raster canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to apply the matrix to the hijack canvas
before attaching the raster canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-04-14 19:17:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-14 19:19:11]

Description was changed from

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to apply the matrix to the hijack canvas
before attaching the raster canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Ensure to reapply a shared canvas transform to image hijack canvas.

When we wrap a shared canvas in an image hijack canvas, it might already
have a transform applied to it. This transform is used to compute the
clip bounds for the canvas. However, since we're wrapping the canvas
into an image hijack canvas, its clip bounds can be different, since
it does not know about the transform on the raster canvas.

The fix to this is to ensure to apply the matrix to the hijack canvas
before attaching the raster canvas.

R=enne
BUG=601913
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7cb1a3b0e0aec2fad7b3f59b789cff12efa3dd12
Cr-Commit-Position: refs/heads/master@{#387390}
==========

[commit-bot: I haz the power, 2016-04-14 19:19:13]

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