Notify RenderView webview() when the device color profile changes

Notify RenderView webview() when the device color profile changes

Notify RenderView webview() when the Page color profile changes so
it can update and repaint all color profiled elements on the Page.

Not used as yet: the IPC from the browser-side will be added later
(need to resolve issue 338130, issue 357443 first), so this change
is just more plumbing.

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
BUG=368663
Committed: https://crrev.com/c76aa4c941af97cb60423c412cae7c7a9b2acd74
Cr-Commit-Position: refs/heads/master@{#296605}

[Noel Gordon, 2014-09-22 15:50:35]

noel@chromium.org changed reviewers:
+ jochen@chromium.org

[Noel Gordon, 2014-09-22 16:16:27]

+dsinclair who was saying something about removing this layer. It's not just
performance, it's the impeadence mismatch of conversion at times too.

[Noel Gordon, 2014-09-22 17:15:02]

https://codereview.chromium.org/592753003/diff/20001/content/renderer/render_...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/592753003/diff/20001/content/renderer/render_...
content/renderer/render_view_impl.cc:3689:
webview()->setDeviceColorProfile(colorProfile);
webview()->setDeviceColorProfile(profile); also works, but the data copy is not
so obvious.

[jochen (gone - plz use gerrit), 2014-09-24 19:58:25]

lgtm

[Noel Gordon, 2014-09-24 23:58:23]

The CQ bit was checked by noel@chromium.org

[commit-bot: I haz the power, 2014-09-24 23:59:22]

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

[commit-bot: I haz the power, 2014-09-25 01:09:03]

Committed patchset #2 (id:20001) as c4957c72a70417bbd9e444aed31b5a73266a404d

[commit-bot: I haz the power, 2014-09-25 01:09:47]

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

[Noel Gordon, 2014-09-30 03:52:48]

https://codereview.chromium.org/592753003/diff/20001/content/renderer/render_...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/592753003/diff/20001/content/renderer/render_...
content/renderer/render_view_impl.cc:3689:
webview()->setDeviceColorProfile(colorProfile);
On 2014/09/22 17:15:02, Noel Gordon wrote:
> webview()->setDeviceColorProfile(profile); also works, but the data copy is
not
> so obvious.

Refer to https://codereview.chromium.org/261633002 to see how this WebVector
gets copied again into WTF::Vector for use inside blink.  Using SharedData<char>
in place of WTF::Vector<char> would save a one copy, but is an unwieldy data
type for the blink sub-system being updated here.