Define ViewMsg_ColorProfile and plumb it into RenderWidget

Define ViewMsg_ColorProfile and plumb it into RenderWidget

Define ViewMsg_ColorProfile IPC message and plumb it into renderer
RenderWidget::SetDeviceColorProfile(). The browser uses the IPC to
set the renderers screen color profile.

Note SetDeviceColorProfile is overridden by RenderViewImpl, so the
IPC message data is handled there - it calls the base RenderWidget
SetDeviceColorProfile, which detects if the color profile changed,
and stores it in the RenderWidget if so.

If there was a change, the RenderViewImpl informs its webview() so
the Page contained therein can update / repaint all color profiled
elements on the Page.

For a diagram of the IPC and procedure call proceedings, including
layout test support (added in issue 369787), see:

  https://crbug.com/368663#c14

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 again just plumbing.

CQ_EXTRA_TRYBOTS=tryserver.blink:mac_blink_rel,mac_blink_dbg
BUG=368663
Committed: https://crrev.com/89949e66232b21fe55eeccd1d2353261de90c275
Cr-Commit-Position: refs/heads/master@{#297328}

[Noel Gordon, 2014-09-25 16:00:09]

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

[Noel Gordon, 2014-09-25 16:00:50]

Add ViewMsg_ColorProfile.

[jochen (gone - plz use gerrit), 2014-09-29 12:51:51]

you'll need an IPC review

btw, I found a thread on chromium-dev "Best way to avoid copying buffers between
WebKit <-> chromium IPC" about avoiding to copy data around too much, and the
conclusion was to use std::vector<char>& in content/chrome, WebData on the API
to blink, and SharedBuffer inside of blink

my bits lgtm

[Noel Gordon, 2014-09-29 14:50:28]

On 2014/09/29 12:51:51, jochen wrote:
> you'll need an IPC review

An IPC review?  Adding jamesr on a punt.

> btw, I found a thread on chromium-dev "Best way to avoid copying buffers
between
> WebKit <-> chromium IPC" about avoiding to copy data around too much, and the
> conclusion was to use std::vector<char>& in content/chrome, WebData on the API
> to blink, and SharedBuffer inside of blink

Noted. Want WTF:Vector in blink in my case :)

> my bits lgtm

Thanks.

[Noel Gordon, 2014-09-29 14:51:02]

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

[Ken Russell (switch to Gerrit), 2014-09-29 20:07:04]

kbr@chromium.org changed reviewers:
+ dcheng@chromium.org, kbr@chromium.org, palmer@chromium.org, wfh@chromium.org

[Ken Russell (switch to Gerrit), 2014-09-29 20:07:05]

+palmer,dcheng,wfh for view_messages.h review

[Will Harris, 2014-09-29 23:35:53]

IPC in content/common/view_messages.h lgtm

[Noel Gordon, 2014-09-30 00:32:49]

The CQ bit was checked by noel@chromium.org

[commit-bot: I haz the power, 2014-09-30 00:33:49]

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

[commit-bot: I haz the power, 2014-09-30 01:12:50]

Committed patchset #1 (id:1) as 883740f54c9d82805471596dd604fd5358131677

[commit-bot: I haz the power, 2014-09-30 01:14:22]

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