Implement input type=color UI

chrome/browser/ui/browser.cc

 // 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.
 
 #include "chrome/browser/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 88 matching lines @@
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/browser/ui/app_modal_dialogs/message_box_handler.h"
 #include "chrome/browser/ui/blocked_content/blocked_content_tab_helper.h"
 #include "chrome/browser/ui/bookmarks/bookmark_tab_helper.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/browser_tab_restore_service_delegate.h"
 #include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/color_chooser.h"
 #include "chrome/browser/ui/constrained_window_tab_helper.h"
 #include "chrome/browser/ui/extensions/shell_window.h"
 #include "chrome/browser/ui/find_bar/find_bar.h"
 #include "chrome/browser/ui/find_bar/find_bar_controller.h"
 #include "chrome/browser/ui/find_bar/find_tab_helper.h"
 #include "chrome/browser/ui/fullscreen_controller.h"
 #include "chrome/browser/ui/global_error.h"
 #include "chrome/browser/ui/global_error_service.h"
 #include "chrome/browser/ui/global_error_service_factory.h"
 #include "chrome/browser/ui/intents/web_intent_picker_controller.h"
@@ skipping 3895 matching lines @@
 
 void Browser::DidNavigateToPendingEntry(WebContents* tab) {
   if (tab == GetSelectedWebContents())
     UpdateBookmarkBarState(BOOKMARK_BAR_STATE_CHANGE_TAB_STATE);
 }
 
 content::JavaScriptDialogCreator* Browser::GetJavaScriptDialogCreator() {
   return GetJavaScriptDialogCreatorInstance();
 }
 
+void Browser::OpenColorChooser(WebContents* tab,
+                               unsigned color_chooser_id,
+                               const SkColor& color) {
+  printf("! Browser::OpenColorChooser %p %d\n", this, color_chooser_id);

[Peter Kasting, 2012/02/07 01:46:24]

No printfs (3 places)

[keishi, 2012/02/17 11:31:05]

Done.

+  if (color_chooser_.get() && color_chooser_id == color_chooser_->identifier())
+    return;
+  if (color_chooser_.get())
+    color_chooser_->End();
+  color_chooser_.reset(ColorChooser::Create(color_chooser_id,
+                                            tab->GetRenderViewHost()));
+  color_chooser_->Open(color);
+}
+void Browser::EndColorChooser(WebContents* tab, unsigned color_chooser_id) {

[Peter Kasting, 2012/02/07 01:46:24]

In general I don't understand why you check both the RVH pointers for equality
AND have the ID checks.  It seems like one must be a superset of the other?  Or
is that not true?

[keishi, 2012/02/17 11:31:05]

The color chooser id is generated by incrementing a counter inside the renderer
process so it overlaps for different renderers. 

+  printf("! Browser::EndColorChooser %p\n", this);
+  if (tab->GetRenderViewHost() != color_chooser_->render_view_host())
+    return;
+  if (tab->GetRenderViewHost() != color_chooser_->render_view_host())

[Peter Kasting, 2012/02/07 01:46:24]

Nit: Two copies of this line, I think you meant to check the ID

[keishi, 2012/02/17 11:31:05]

Done.

+    return;
+  color_chooser_->End();
+}
+void Browser::SetSelectedColorInColorChooser(WebContents* tab,
+    unsigned color_chooser_id,

[Peter Kasting, 2012/02/07 01:46:24]

Nit: Indent even.  If it won't fit, indent even the first arg 4

[keishi, 2012/02/17 11:31:05]

Done.

+    const SkColor& color) {
+  if (!color_chooser_.get())

[Peter Kasting, 2012/02/07 01:46:24]

Nit: Combine these conditionals

[keishi, 2012/02/17 11:31:05]

Done.

+    return;
+  if (tab->GetRenderViewHost() != color_chooser_->render_view_host())
+    return;
+  if (color_chooser_id != color_chooser_->identifier())
+    return;
+  color_chooser_->SetSelectedColor(color);
+}
+void Browser::DidEndColorChooser(WebContents* tab) {
+  printf("! Browser::DidEndColorChooser %p\n", this);
+  if (tab->GetRenderViewHost() != color_chooser_->render_view_host())
+    return;
+  color_chooser_.reset();
+}
+
 void Browser::RunFileChooser(WebContents* tab,
                              const content::FileChooserParams& params) {
   RunFileChooserHelper(tab, params);
 }
 
 void Browser::EnumerateDirectory(WebContents* tab, int request_id,
                                  const FilePath& path) {
   EnumerateDirectoryHelper(tab, request_id, path);
 }
 
@@ skipping 1472 matching lines @@
           profile()->GetOriginalProfile());
   if (service->HasSyncSetupCompleted())
     ShowOptionsTab(chrome::kPersonalOptionsSubPage);
   else
     service->ShowLoginDialog();
 }
 
 void Browser::ToggleSpeechInput() {
   GetSelectedWebContents()->GetRenderViewHost()->ToggleSpeechInput();
 }