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..1bd7613a0e23f6f588a22b244060fdba6aaa3956
--- /dev/null
+++ b/chrome/browser/ui/color_chooser.h
@@ -0,0 +1,22 @@
+// 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 CHROME_BROWSER_UI_COLOR_CHOOSER_H_
+#define CHROME_BROWSER_UI_COLOR_CHOOSER_H_
+#pragma once
+
+#include "content/public/browser/color_chooser.h"
+
+namespace content {
+class WebContents;
+}
+
+// Abstraction object for color choosers for each platforms.
+class ColorChooser : public content::ColorChooser {

[Peter Kasting, 2012/02/29 01:40:07]

Why do you need this second abstraction layer?  Why can't the Create() function
here be declared in the content header with the rest of content::ColorChooser,
and eliminate this class entirely?

[keishi, 2012/02/29 13:05:22]

I was thinking that too. Removed ColorChooser class.

+ public:
+  static ColorChooser* Create(
+      int identifier, content::WebContents* tab, SkColor initial_color);
+};
+
+#endif  // CHROME_BROWSER_UI_COLOR_CHOOSER_H_