Make OffscreenCanvas commit() propagate the origin-clean flag.

Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas. The placeholder code was modified to
use StaticBitmapImage instead of Image to gain access to the
originClean() method without downcasting.

BUG=673348
Committed: https://crrev.com/0eea57f836fb2551563e1f18d5e91215bf0f5150
Cr-Commit-Position: refs/heads/master@{#438172}

[Justin Novosad, 2016-12-12 21:20:39]

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

[Justin Novosad, 2016-12-12 21:20:50]

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

[Justin Novosad, 2016-12-12 21:20:51]

PTAL

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

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-12 21:23:17]

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

[commit-bot: I haz the power, 2016-12-12 21:23:41]

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

[Stephen White, 2016-12-12 22:19:51]

LGTM

Might want to explain the change from Image to StaticBitmapImage in the CL
description.

[Justin Novosad, 2016-12-12 22:42:38]

Description was changed from

==========
Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas.

BUG=673348
==========

to

==========
Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas. The placeholder code was modified to
use StaticBitmapImage instead of Image to gain access to the
originClean() method without downcasting.

BUG=673348
==========

[Justin Novosad, 2016-12-12 22:42:45]

The CQ bit was unchecked by junov@chromium.org

[Justin Novosad, 2016-12-12 22:42:49]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-12-12 22:45:06]

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-13 01:50:35]

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

[commit-bot: I haz the power, 2016-12-13 01:50:36]

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

[Justin Novosad, 2016-12-13 14:53:39]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-12-13 14:53:56]

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-13 15:25:51]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481640819000880,
"parent_rev": "cf13e14559c36d0d62b0413b3b5a8b35704f31bc", "commit_rev":
"6bbac82231faf27ab64cbd73df6e7e047225984d"}

[commit-bot: I haz the power, 2016-12-13 15:26:17]

Description was changed from

==========
Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas. The placeholder code was modified to
use StaticBitmapImage instead of Image to gain access to the
originClean() method without downcasting.

BUG=673348
==========

to

==========
Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas. The placeholder code was modified to
use StaticBitmapImage instead of Image to gain access to the
originClean() method without downcasting.

BUG=673348

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

[commit-bot: I haz the power, 2016-12-13 15:26:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 15:28:56]

Description was changed from

==========
Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas. The placeholder code was modified to
use StaticBitmapImage instead of Image to gain access to the
originClean() method without downcasting.

BUG=673348

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

to

==========
Make OffscreenCanvas commit() propagate the origin-clean flag.

This change block user code from accessing cross-origin image data
from a placeholder canvas. The placeholder code was modified to
use StaticBitmapImage instead of Image to gain access to the
originClean() method without downcasting.

BUG=673348

Committed: https://crrev.com/0eea57f836fb2551563e1f18d5e91215bf0f5150
Cr-Commit-Position: refs/heads/master@{#438172}
==========

[commit-bot: I haz the power, 2016-12-13 15:28:57]

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