Implement input type=color UI

chrome/browser/ui/color_chooser.h

Index: chrome/browser/ui/color_chooser.h
diff --git a/chrome/browser/ui/color_chooser.h b/chrome/browser/ui/color_chooser.h
new file mode 100644
index 0000000000000000000000000000000000000000..4f79343b342a1e7ab9363363929d4b4c1a22e90c
--- /dev/null
+++ b/chrome/browser/ui/color_chooser.h
@@ -0,0 +1,34 @@
+// Copyright (c) 2011 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 CHROME_BROWSER_UI_COLOR_CHOOSER_H_
+#define CHROME_BROWSER_UI_COLOR_CHOOSER_H_
+#pragma once
+
+#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebColor.h"
+#include "ui/gfx/native_widget_types.h"
+
+class RenderViewHost;
+
+// Abstraction object for color choosers of each platforms.
+class ColorChooser {
+ public:
+  // An interface implemented by a Listener object wishing to know about the
+  // the results from the color chooser.
+  class Listener {
+   public:
+    virtual void DidChooseColor(WebKit::WebColor color) = 0;

[Peter Kasting, 2012/01/27 18:27:15]

Don't use WebKit:: types in Chrome code.  Use SkColor, for which platform-native
color conversion functions already exist.

[keishi, 2012/02/06 15:01:57]

Done.

+    virtual void DidEnd() = 0;
+    virtual RenderViewHost* GetRenderViewHost() = 0;
+  };
+
+  static ColorChooser* Create();

[Peter Kasting, 2012/01/27 18:27:15]

Why not have this function take a listener, instead of Open() and End(), and
have the concrete class store this as the listener object?  Seems like this
would also eliminate the need for your ColorSelectHelper object.

[keishi, 2012/02/06 15:01:57]

Done.

+  virtual ~ColorChooser() {}
+
+  virtual void Open(Listener* listener, WebKit::WebColor initial_color) = 0;
+  virtual void End(Listener* listener) = 0;
+  virtual void SetSelectedColor(WebKit::WebColor color) = 0;
+};
+
+#endif  // CHROME_BROWSER_UI_COLOR_CHOOSER_H_