Notify RenderView webview() when the device color profile changes

content/renderer/render_view_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 3667 matching lines @@
     AutoResizeCompositor();
 
   if (browser_plugin_manager_.get())
     browser_plugin_manager_->UpdateDeviceScaleFactor();
 }
 
 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;
 }
 
 void RenderViewImpl::ResetDeviceColorProfileForTesting() {
   RenderWidget::ResetDeviceColorProfileForTesting();
   if (webview())
     webview()->resetDeviceColorProfile();
 }
 
@@ skipping 516 matching lines @@
     std::vector<gfx::Size> sizes;
     ConvertToFaviconSizes(icon_urls[i].sizes(), &sizes);
     if (!url.isEmpty())
       urls.push_back(
           FaviconURL(url, ToFaviconType(icon_urls[i].iconType()), sizes));
   }
   SendUpdateFaviconURL(urls);
 }
 
 }  // namespace content