Allow specifying ColorBehavior to ImageSource

Allow specifying ColorBehavior to ImageSource

This patch has no functional change, but will allow subsequent patches
to specify a different target color space depending on their application
(e.g, canvas and WebGL can request sRGB, and raster can request
the output device behavior).

BUG=667420
Committed: https://crrev.com/80eae1dfec8262cb7cc2f585d85d1d41a63a1168
Cr-Commit-Position: refs/heads/master@{#437514}

[ccameron, 2016-12-07 00:28:39]

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

[commit-bot: I haz the power, 2016-12-07 00:29:43]

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-12-07 00:56:28]

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

[commit-bot: I haz the power, 2016-12-07 00:56:29]

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

[ccameron, 2016-12-07 08:03:58]

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

[commit-bot: I haz the power, 2016-12-07 08:04:16]

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-12-07 10:57:55]

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

[commit-bot: I haz the power, 2016-12-07 10:57:56]

Dry run: This issue passed the CQ dry run.

[ccameron, 2016-12-07 23:49:06]

Description was changed from

==========
Allow specifying ColorBehavior to ImageSource

BUG=667420
==========

to

==========
Allow specifying ColorBehavior to ImageSource

This patch has no functional change, but will allow subsequent patches
to specify a different target color space depending on their application
(e.g, canvas and WebGL can request sRGB, and raster can request
the output device behavior).

BUG=667420
==========

[ccameron, 2016-12-07 23:49:07]

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

[ccameron, 2016-12-07 23:50:37]

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

[commit-bot: I haz the power, 2016-12-07 23:51:05]

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-12-08 01:19:03]

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

[commit-bot: I haz the power, 2016-12-08 01:19:04]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[ccameron, 2016-12-08 01:25:15]

ptal -- fairly simple change. The bigger ones come next.

[ccameron, 2016-12-08 02:22:11]

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

[commit-bot: I haz the power, 2016-12-08 02:22:56]

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-12-08 03:52:58]

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

[commit-bot: I haz the power, 2016-12-08 03:52:59]

Dry run: This issue passed the CQ dry run.

[ccameron, 2016-12-08 20:04:52]

ping on this

[ccameron, 2016-12-08 20:57:50]

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

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

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-09 02:23:56]

lgtm

[ccameron, 2016-12-09 03:31:55]

The CQ bit was checked by ccameron@chromium.org

[commit-bot: I haz the power, 2016-12-09 03:32:24]

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-09 05:09:21]

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

[commit-bot: I haz the power, 2016-12-09 05:09:23]

Failed to apply patch for
third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in:
While running git apply --index -p1;
  error: patch failed:
third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in:242
  error: third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in: patch
does not apply

Patch:       third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
Index: third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
diff --git a/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
b/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
index
dc1ae5a5e08376f4e55ada1b977a9d0d9b41f686..f74272430f6395b99996fa8cb80f310fb5b0e596
100644
--- a/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
+++ b/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
@@ -242,7 +242,7 @@ TimerThrottlingForHiddenFrames status=stable
 // enabled since they're a feature always supported by Chrome.
 TouchEventAPI status=stable
 TraceWrappables
-TrueColorRendering status=experimental, settable_from_internals=True
+TrueColorRendering status=experimental
 TrustedEventsDefaultAction status=stable
 UserSelectAll status=stable
 WebAnimationsAPI status=experimental

[ccameron, 2016-12-09 09:17:17]

The CQ bit was checked by ccameron@chromium.org

[ccameron, 2016-12-09 09:17:18]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org
Link to the patchset: https://codereview.chromium.org/2556973002/#ps140001
(title: "Rebase")

[commit-bot: I haz the power, 2016-12-09 09:17:34]

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-09 10:49:17]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1481275037527430,
"parent_rev": "4b7c308b1b7a4e880f9df00f442025d24c01131b", "commit_rev":
"1197610542c8e8426c86737e381f160c1d925c1f"}

[commit-bot: I haz the power, 2016-12-09 10:49:42]

Description was changed from

==========
Allow specifying ColorBehavior to ImageSource

This patch has no functional change, but will allow subsequent patches
to specify a different target color space depending on their application
(e.g, canvas and WebGL can request sRGB, and raster can request
the output device behavior).

BUG=667420
==========

to

==========
Allow specifying ColorBehavior to ImageSource

This patch has no functional change, but will allow subsequent patches
to specify a different target color space depending on their application
(e.g, canvas and WebGL can request sRGB, and raster can request
the output device behavior).

BUG=667420

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

[commit-bot: I haz the power, 2016-12-09 10:49:43]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-12-09 10:51:16]

Description was changed from

==========
Allow specifying ColorBehavior to ImageSource

This patch has no functional change, but will allow subsequent patches
to specify a different target color space depending on their application
(e.g, canvas and WebGL can request sRGB, and raster can request
the output device behavior).

BUG=667420

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

to

==========
Allow specifying ColorBehavior to ImageSource

This patch has no functional change, but will allow subsequent patches
to specify a different target color space depending on their application
(e.g, canvas and WebGL can request sRGB, and raster can request
the output device behavior).

BUG=667420

Committed: https://crrev.com/80eae1dfec8262cb7cc2f585d85d1d41a63a1168
Cr-Commit-Position: refs/heads/master@{#437514}
==========

[commit-bot: I haz the power, 2016-12-09 10:51:17]

Patchset 8 (id:??) landed as
https://crrev.com/80eae1dfec8262cb7cc2f585d85d1d41a63a1168
Cr-Commit-Position: refs/heads/master@{#437514}