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

chrome/browser/ui/color_chooser_util.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.

[tfarina, 2013/05/20 16:45:53]

Copyright 2013

+// 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_UTIL_H_

[tfarina, 2013/05/20 16:45:53]

I'd name this file color_chooser_dialog.h

+#define CHROME_BROWSER_UI_COLOR_CHOOSER_UTIL_H_
+
+#include "third_party/skia/include/core/SkColor.h"
+
+namespace content {
+class WebContents;
+class ColorChooser;

[tfarina, 2013/05/20 16:45:53]

sort

+}
+
+content::ColorChooser* OpenColorChooser(content::WebContents* web_contents,

[Ben Goodger (Google), 2013/05/20 16:14:30]

1. put this in browser/ui/browser_dialogs.h instead of making a new file.
2. rename this method ShowColorChooser(..) like the other methods in
browser_dialogs. This will make it so you won't have a function name that
collides with the WCD method.

[tfarina, 2013/05/20 16:45:53]

I'd also put this in chrome namespace. I don't recall if browser_dialogs.h is
already in that namespace.

[keishi, 2013/05/21 16:48:06]

Done. Thanks, this looks like the perfect place to put it.

+                                        SkColor initial_color);
+
+#endif  // CHROME_BROWSER_UI_COLOR_CHOOSER_UTIL_H_