Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5293)

Unified Diff: chrome/browser/ui/browser.cc

Issue 13150004: Support color chooser inside extesions, apps, chrome frame, dev tool (Closed) Base URL: http://git.chromium.org/chromium/src.git@ngcolor
Patch Set: Removed chrome/browser/ui/color_chooser.cc Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index e6f17f668c68bd0d8673fe64ffbc89e4dc2ff012..22381e60da429465aff98c770aa5be32d5bdee71 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1522,27 +1522,8 @@ content::JavaScriptDialogManager* Browser::GetJavaScriptDialogManager() {
}
content::ColorChooser* Browser::OpenColorChooser(WebContents* web_contents,
- int color_chooser_id,
- SkColor color) {
-#if defined(OS_WIN)
- // On Windows, only create a color chooser if one doesn't exist, because we
- // can't close the old color chooser dialog.
- if (!color_chooser_.get())
- color_chooser_.reset(content::ColorChooser::Create(color_chooser_id,
- web_contents,
- color));
-#else
- if (color_chooser_.get())
- color_chooser_->End();
- color_chooser_.reset(content::ColorChooser::Create(color_chooser_id,
- web_contents,
- color));
-#endif
- return color_chooser_.get();
-}
-
-void Browser::DidEndColorChooser() {
- color_chooser_.reset();
+ SkColor initial_color) {
+ return content::ColorChooser::Open(web_contents, initial_color);
}
void Browser::RunFileChooser(WebContents* web_contents,

Powered by Google App Engine
This is Rietveld 408576698