Notify RenderView webview() when the device color profile changes

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index eba0382f3453be34ace76c5ecbc5423274f14451..a5f404ba3a3d63a05e8981cae22fe986fdaa89c3 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -3685,8 +3685,8 @@ bool RenderViewImpl::SetDeviceColorProfile(
     const std::vector<char>& profile) {
   bool changed = RenderWidget::SetDeviceColorProfile(profile);
   if (changed && webview()) {
-    // TODO(noel): notify the webview() of the color profile change so it
-    // can update and repaint all color profiled page elements.
+    WebVector<char> colorProfile = profile;
+    webview()->setDeviceColorProfile(colorProfile);

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

webview()->setDeviceColorProfile(profile); also works, but the data copy is not
so obvious.

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

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.

   }
   return changed;
 }