Change Panel titlebars to activate the panel on click (rather than minimize).

chrome/browser/ui/panels/panel_browser_view.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/panels/panel_browser_view.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "chrome/browser/native_window_notification_source.h"
 #include "chrome/browser/ui/panels/display_settings_provider.h"
@@ skipping 10 matching lines @@
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/widget/widget.h"
 
 #if defined(OS_WIN) && !defined(USE_AURA)
 #include "base/win/win_util.h"  // for IsCtrlPressed()
 #endif
 
 using content::WebContents;
 
-namespace {
-// The threshold to differentiate the short click and long click.
-const int kShortClickThresholdMs = 200;
-
-// Delay before click-to-minimize is allowed after the attention has been
-// cleared.
-const int kSuspendMinimizeOnClickIntervalMs = 500;
-}
-
 NativePanel* Panel::CreateNativePanel(Browser* browser, Panel* panel,
                                       const gfx::Rect& bounds) {
   PanelBrowserView* view = new PanelBrowserView(browser, panel, bounds);
   (new BrowserFrame(view))->InitBrowserFrame();
   return view;
 }
 
 PanelBrowserView::PanelBrowserView(Browser* browser, Panel* panel,
                                    const gfx::Rect& bounds)
   : BrowserView(browser),
@@ skipping 130 matching lines @@
       GetFrameView()->GetWidget()->GetNativeView() == ::GetForegroundWindow();
 #else
   NOTIMPLEMENTED();
   bool focused = active;
 #endif
 
   if (focused_ == focused)
     return;
   focused_ = focused;
 
-  if (focused_) {
-    // Expand the panel if needed. Do NOT expand a TITLE_ONLY panel
-    // otherwise it will be impossible to drag a title without
-    // expanding it.
-    if (panel_->expansion_state() == Panel::MINIMIZED)
-      panel_->SetExpansionState(Panel::EXPANDED);
-
-    if (is_drawing_attention_) {
-      panel_->FlashFrame(false);
-
-      // Restore the panel from title-only mode here. Could not do this in the
-      // code above.
-      if (panel_->expansion_state() == Panel::TITLE_ONLY)
-        panel_->SetExpansionState(Panel::EXPANDED);
-
-      // This function is called per one of the following user interactions:
-      // 1) clicking on the title-bar
-      // 2) clicking on the client area
-      // 3) switching to the panel via keyboard
-      // For case 1, we do not want the expanded panel to be minimized since the
-      // user clicks on it to mean to clear the attention.
-      attention_cleared_time_ = base::TimeTicks::Now();
-    }
-  }
+  // Do not clear draw attention if user cannot see contents of panel.
+  if (focused_ && is_drawing_attention_ && !panel_->IsMinimized())
+    panel_->FlashFrame(false);
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_PANEL_CHANGED_ACTIVE_STATUS,
       content::Source<Panel>(panel()),
       content::NotificationService::NoDetails());
 }
 
 bool PanelBrowserView::AcceleratorPressed(
     const ui::Accelerator& accelerator) {
   if (mouse_pressed_ && accelerator.key_code() == ui::VKEY_ESCAPE) {
@@ skipping 299 matching lines @@
     }
 
     return EndDragging(false);
   }
 
   // If the panel drag was cancelled before the mouse is released, do not treat
   // this as a click.
   if (mouse_dragging_state_ != NO_DRAGGING)
     return true;
 
-  // Ignore long clicks. Treated as a canceled click to be consistent with Mac.
-  if (base::TimeTicks::Now() - mouse_pressed_time_ >
-      base::TimeDelta::FromMilliseconds(kShortClickThresholdMs))
-    return true;
-
+  panel::ClickModifier click_modifier = panel::NO_MODIFIER;
 #if defined(OS_WIN) && !defined(USE_AURA)
   if (base::win::IsCtrlPressed()) {

[jianli, 2012/04/18 22:18:47]

nit: {} is not needed.

-    panel_->OnTitlebarClicked(panel::APPLY_TO_ALL);
-    return true;
+    click_modifier = panel::APPLY_TO_ALL;

[jianli, 2012/04/18 22:18:47]

Since we're not longer supporting click-to-minimize, do we want also change the
pattern for minimize-all? Like, holding ctrl while clicking on minimize button?

   }
 #else
-  NOTIMPLEMENTED();  // Proceed without modifier.
+  // Proceed without modifier.
 #endif
 
-  // TODO(jennb): Move remaining titlebar click handling out of here.
-  // (http://crbug.com/118431)
-  PanelStrip* panel_strip = panel_->panel_strip();
-  if (!panel_strip)
-    return true;
-
-  // Do not minimize the panel when we just clear the attention state. This is
-  // a hack to prevent the panel from being minimized when the user clicks on
-  // the title-bar to clear the attention.
-  if (panel_strip->type() == PanelStrip::DOCKED &&
-      panel_->expansion_state() == Panel::EXPANDED &&
-      base::TimeTicks::Now() - attention_cleared_time_ <
-      base::TimeDelta::FromMilliseconds(kSuspendMinimizeOnClickIntervalMs)) {
-    return true;
-  }
-
-  if (panel_strip->type() == PanelStrip::DOCKED &&
-      panel_->expansion_state() == Panel::EXPANDED)
-    panel_->SetExpansionState(Panel::MINIMIZED);
-  else
-    panel_->Activate();
-
+  panel_->OnTitlebarClicked(click_modifier);
   return true;
 }
 
 bool PanelBrowserView::OnTitlebarMouseCaptureLost() {
   if (mouse_dragging_state_ == DRAGGING_STARTED)
     return EndDragging(true);
   return true;
 }
 
 bool PanelBrowserView::EndDragging(bool cancelled) {
@@ skipping 163 matching lines @@
   return expected_color == frame_view->title_label_->enabled_color();
 }
 
 bool NativePanelTestingWin::IsWindowSizeKnown() const {
   return true;
 }
 
 bool NativePanelTestingWin::IsAnimatingBounds() const {
   return panel_browser_view_->IsAnimatingBounds();
 }