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

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

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: Rebased Created 7 years, 7 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.h
diff --git a/chrome/browser/ui/browser.h b/chrome/browser/ui/browser.h
index de9d1e19c4f14d714626a03546f526f24330d120..816b78acccefc07ec9fb54afc8c198433cbe974b 100644
--- a/chrome/browser/ui/browser.h
+++ b/chrome/browser/ui/browser.h
@@ -582,10 +582,7 @@ class Browser : public TabStripModelObserver,
virtual content::JavaScriptDialogManager*
GetJavaScriptDialogManager() OVERRIDE;
virtual content::ColorChooser* OpenColorChooser(
- content::WebContents* web_contents,
- int color_chooser_id,
- SkColor color) OVERRIDE;
- virtual void DidEndColorChooser() OVERRIDE;
+ content::WebContents* web_contents, SkColor color) OVERRIDE;
virtual void RunFileChooser(
content::WebContents* web_contents,
const content::FileChooserParams& params) OVERRIDE;
@@ -916,10 +913,6 @@ class Browser : public TabStripModelObserver,
// True if the browser window has been shown at least once.
bool window_has_shown_;
- // Currently open color chooser. Non-NULL after OpenColorChooser is called and
- // before DidEndColorChooser is called.
- scoped_ptr<content::ColorChooser> color_chooser_;
-
DISALLOW_COPY_AND_ASSIGN(Browser);
};

Powered by Google App Engine
This is Rietveld 408576698