Support color chooser inside extesions, apps, chrome frame, dev tool

chrome/browser/ui/color_chooser.h

+// Copyright 2013 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_
+
+#include "base/basictypes.h"
+#include "content/public/browser/color_chooser.h"
+#include "third_party/skia/include/core/SkColor.h"
+
+class ColorChooser : public content::ColorChooser {
+ public:
+  static ColorChooser* get_current_color_chooser() {
+    return current_color_chooser_;
+  }
+
+  // Returns NULL on failure.

[Ben Goodger (Google), 2013/04/16 16:20:17]

I left you a comment earlier about this. It's not clear why this method would
ever "fail". You should just refocus the existing one. For that matter, it's
probably not necessary to expose the current color chooser above. Just having
this single method that opens a new one or refocuses the existing should be
sufficient.

[keishi, 2013/04/17 10:49:48]

I'm sorry I didn't understand what you were getting at earlier but you are
right, I don't need to return NULL.

+  static ColorChooser* Open(content::WebContents* web_contents,
+                            SkColor initial_color);
+
+  explicit ColorChooser(content::WebContents* web_contents);

[Ben Goodger (Google), 2013/04/16 16:20:17]

ctor/dtor should be protected.

[keishi, 2013/04/17 10:49:48]

Done.

+  virtual ~ColorChooser();
+
+ protected:
+  void DidChooseColor(SkColor color);
+  void DidEndColorChooser();
+
+ private:
+  static ColorChooser* current_color_chooser_;
+
+  // The web contents invoking the color chooser.  No ownership because it will
+  // outlive this class.
+  content::WebContents* web_contents_;
+
+  DISALLOW_COPY_AND_ASSIGN(ColorChooser);
+};
+
+#endif  // CHROME_BROWSER_UI_COLOR_CHOOSER_H_