Create ImageDecoder target color space

third_party/WebKit/Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 3027 matching lines @@
     float zoomFactorForDeviceScaleFactor) {
   m_zoomFactorForDeviceScaleFactor = zoomFactorForDeviceScaleFactor;
   if (!m_layerTreeView)
     return;
   setZoomLevel(m_zoomLevel);
 }
 
 void WebViewImpl::setDeviceColorProfile(const WebVector<char>& colorProfile) {
   Vector<char> deviceProfile;
   deviceProfile.append(colorProfile.data(), colorProfile.size());
-  ImageDecoder::setTargetColorProfile(deviceProfile);
+  ImageDecoder::setGlobalTargetColorProfile(deviceProfile);

[esprehn, 2016/11/21 20:19:30]

This is forcing a copy to convert from WTF::Vector to WebVector here. The
setGlobalTargetColorProfile method should really take a Vector, or just pass the
colorProfile  variable directly?

[ccameron, 2016/11/21 22:28:14]

Updated to pass colorProfile directly (I wonder why it wasn't...).

 }
 
 void WebViewImpl::enableAutoResizeMode(const WebSize& minSize,
                                        const WebSize& maxSize) {
   m_shouldAutoResize = true;
   m_minAutoSize = minSize;
   m_maxAutoSize = maxSize;
   configureAutoResizeMode();
 }
 
@@ skipping 1158 matching lines @@
   if (focusedFrame->localFrameRoot() != mainFrameImpl()->frame())
     return nullptr;
   return focusedFrame;
 }
 
 LocalFrame* WebViewImpl::focusedLocalFrameAvailableForIme() const {
   return m_imeAcceptEvents ? focusedLocalFrameInWidget() : nullptr;
 }
 
 }  // namespace blink