Implement input type=color UI

chrome/browser/ui/browser.h

 // 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_BROWSER_H_
 #define CHROME_BROWSER_UI_BROWSER_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 37 matching lines @@
 #include "content/public/browser/page_navigator.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/common/page_zoom.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/gfx/rect.h"
 
 class BrowserSyncedWindowDelegate;
 class BrowserTabRestoreServiceDelegate;
 class BrowserWindow;
+class ColorChooser;
 class Extension;
 class FindBarController;
 class FullscreenController;
 class HtmlDialogUIDelegate;
 class InstantController;
 class InstantUnloadHandler;
 class PrefService;
 class Profile;
 class SkBitmap;
 class StatusBubble;
@@ skipping 939 matching lines @@
   virtual void ContentRestrictionsChanged(
       content::WebContents* source) OVERRIDE;
   virtual void RendererUnresponsive(content::WebContents* source) OVERRIDE;
   virtual void RendererResponsive(content::WebContents* source) OVERRIDE;
   virtual void WorkerCrashed(content::WebContents* source) OVERRIDE;
   virtual void DidNavigateMainFramePostCommit(
       content::WebContents* tab) OVERRIDE;
   virtual void DidNavigateToPendingEntry(content::WebContents* tab) OVERRIDE;
   virtual content::JavaScriptDialogCreator*
       GetJavaScriptDialogCreator() OVERRIDE;
+  virtual content::ColorChooser* OpenColorChooser(content::WebContents* tab,
+                                int color_chooser_id,

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

Nit: Aligned with above arg, or all args (including first) indented 4 if that
won't fit

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

Done.

+                                const SkColor& color) OVERRIDE;
+    virtual void DidEndColorChooser() OVERRIDE;

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

Nit: Indent 2, not 4

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

Done.

   virtual void RunFileChooser(
       content::WebContents* tab,
       const content::FileChooserParams& params) OVERRIDE;
   virtual void EnumerateDirectory(content::WebContents* tab, int request_id,
                                   const FilePath& path) OVERRIDE;
   virtual void ToggleFullscreenModeForTab(content::WebContents* tab,
       bool enter_fullscreen) OVERRIDE;
   virtual bool IsFullscreenForTab(
       const content::WebContents* tab) const OVERRIDE;
   virtual void JSOutOfMemory(content::WebContents* tab) OVERRIDE;
@@ skipping 426 matching lines @@
   scoped_ptr<InstantController> instant_;
   scoped_ptr<InstantUnloadHandler> instant_unload_handler_;
 
   BookmarkBar::State bookmark_bar_state_;
 
   scoped_refptr<FullscreenController> fullscreen_controller_;
 
   // True if the browser window has been shown at least once.
   bool window_has_shown_;
 
+  scoped_ptr<ColorChooser> color_chooser_;

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

Nit: Add description (e.g. mentioning when this would be non-NULL)

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

Done.

+
   DISALLOW_COPY_AND_ASSIGN(Browser);
 };
 
 #endif  // CHROME_BROWSER_UI_BROWSER_H_