Teach gfx::Screen about color profiles

ui/gfx/screen.h

 // 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.
 
 #ifndef UI_GFX_SCREEN_H_
 #define UI_GFX_SCREEN_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 55 matching lines @@
   // Mirrored displays are excluded; this method is intended to return the
   // number of distinct, usable displays.
   virtual int GetNumDisplays() const = 0;
 
   // Returns the list of displays that are currently available.
   virtual std::vector<gfx::Display> GetAllDisplays() const = 0;
 
   // Returns the display nearest the specified window.
   virtual gfx::Display GetDisplayNearestWindow(NativeView view) const = 0;
 
+  // Returns the ICC color profile data of the display nearest the window.
+  virtual bool GetDisplayColorProfile(
+      NativeView view, std::vector<char>* color_profile) const = 0;

[sky, 2014/05/07 16:38:35]

I agree with Oshima on the wierdness of this API. What is NativeView isn't on
this screen at all?

What if view is NULL?

Also, when you wrap one param per line (see style guide).

+
   // Returns the display nearest the specified point.
   virtual gfx::Display GetDisplayNearestPoint(
       const gfx::Point& point) const = 0;
 
   // Returns the display that most closely intersects the provided bounds.
   virtual gfx::Display GetDisplayMatching(
       const gfx::Rect& match_rect) const = 0;
 
   // Returns the primary display.
   virtual gfx::Display GetPrimaryDisplay() const = 0;
 
   // Adds/Removes display observers.
   virtual void AddObserver(DisplayObserver* observer) = 0;
   virtual void RemoveObserver(DisplayObserver* observer) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Screen);
 };
 
 Screen* CreateNativeScreen();
 
 }  // namespace gfx
 
 #endif  // UI_GFX_SCREEN_H_