Finalizing the 'OnClick' behavior of the Launcher items.

ash/launcher/launcher_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 "ash/launcher/launcher_view.h"
 
 #include <algorithm>
 
 #include "ash/ash_constants.h"
 #include "ash/ash_switches.h"
@@ skipping 13 matching lines @@
 #include "base/memory/scoped_ptr.h"
 #include "grit/ash_strings.h"
 #include "grit/ash_resources.h"
 #include "ui/aura/window.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/models/simple_menu_model.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/compositor/layer.h"
 #include "ui/compositor/layer_animator.h"
 #include "ui/gfx/canvas.h"
+#include "ui/native_theme/native_theme.h"
 #include "ui/views/animation/bounds_animator.h"
 #include "ui/views/border.h"
 #include "ui/views/controls/menu/menu_model_adapter.h"
 #include "ui/views/controls/menu/menu_runner.h"
 #include "ui/views/focus/focus_search.h"
 #include "ui/views/focus_border.h"
 #include "ui/views/view_model.h"
 #include "ui/views/view_model_utils.h"
 #include "ui/views/widget/widget.h"
 
@@ skipping 16 matching lines @@
 const int kHorizontalIconSpacing = 8;
 
 // The proportion of the launcher space reserved for non-panel icons. Panels
 // may flow into this space but will be put into the overflow bubble if there
 // is contention for the space.
 const float kReservedNonPanelIconProportion = 0.67f;
 
 // This is the command id of the menu item which contains the name of the menu.
 const int kCommandIdOfMenuName = 0;
 
+// This is the command id of the active menu item.
+const int kCommandIdOfActiveName = 1;
+
 namespace {
 
 // The MenuModelAdapter gets slightly changed to adapt the menu appearance to
 // our requirements.
 class LauncherMenuModelAdapter
     : public views::MenuModelAdapter {
  public:
   explicit LauncherMenuModelAdapter(ui::MenuModel* menu_model);
 
   // Overriding MenuModelAdapter's MenuDelegate implementation.
   virtual const gfx::Font* GetLabelFont(int command_id) const OVERRIDE;
   virtual void GetHorizontalIconMargins(int id,
                                         int icon_size,
                                         int* left_margin,
                                         int* right_margin) const OVERRIDE;
-
+  virtual bool GetBackgroundColor(int command_id,
+                                  bool is_hovered,
+                                  SkColor* override_color) const OVERRIDE;
  private:
 
   DISALLOW_COPY_AND_ASSIGN(LauncherMenuModelAdapter);
 };
 
 
 LauncherMenuModelAdapter::LauncherMenuModelAdapter(ui::MenuModel* menu_model)
     : MenuModelAdapter(menu_model) {}
 
 const gfx::Font* LauncherMenuModelAdapter::GetLabelFont(
     int command_id) const {
   if (command_id != kCommandIdOfMenuName)
     return MenuModelAdapter::GetLabelFont(command_id);
 
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   return &rb.GetFont(ui::ResourceBundle::BoldFont);
 }
 
+bool LauncherMenuModelAdapter::GetBackgroundColor(
+    int command_id,
+    bool is_hovered,
+    SkColor *override_color) const {
+  if (command_id != kCommandIdOfActiveName)
+    return false;
+
+  ui::NativeTheme* native_theme = ui::NativeTheme::instance();
+  *override_color = native_theme->GetSystemColor(is_hovered ?
+      ui::NativeTheme::kColorId_FocusedSelectedMenuItemBackground :
+      ui::NativeTheme::kColorId_SelectedMenuItemBackground);
+  return true;
+}
+
 void LauncherMenuModelAdapter::GetHorizontalIconMargins(
     int command_id,
     int icon_size,
     int* left_margin,
     int* right_margin) const {
   *left_margin = kHorizontalIconSpacing;
   *right_margin = (command_id != kCommandIdOfMenuName) ?
       kHorizontalIconSpacing : -icon_size;
 }
 
@@ skipping 1068 matching lines @@
     return;
   }
 
   int view_index = view_model_->GetIndexOfView(sender);
   // May be -1 while in the process of animating closed.
   if (view_index == -1)
     return;
 
   tooltip_->Close();
 
+  if (event.IsShiftDown())

[sky, 2013/01/31 21:15:02]

This code is fragile (I know I've added some too), can you create a scoping
object whose constructor turns it on and destructor unsets.

[Mr4D (OOO till 08-26), 2013/01/31 22:38:52]

Done.

+    ui::LayerAnimator::set_slow_animation_mode(true);
+
   // Collect usage statistics before we decide what to do with the click.
   switch (model_->items()[view_index].type) {
     case TYPE_APP_SHORTCUT:
     case TYPE_PLATFORM_APP:
       Shell::GetInstance()->delegate()->RecordUserMetricsAction(
           UMA_LAUNCHER_CLICK_ON_APP);
+      delegate_->ItemClicked(model_->items()[view_index], event.flags());
+      // Fallthrough
+    case TYPE_TABBED:
+    case TYPE_APP_PANEL:
       break;
 
     case TYPE_APP_LIST:
       Shell::GetInstance()->delegate()->RecordUserMetricsAction(
           UMA_LAUNCHER_CLICK_ON_APPLIST_BUTTON);
+      Shell::GetInstance()->ToggleAppList(GetWidget()->GetNativeView());
       break;
 
     case TYPE_BROWSER_SHORTCUT:
       // Click on browser icon is counted in app clicks.
       Shell::GetInstance()->delegate()->RecordUserMetricsAction(
           UMA_LAUNCHER_CLICK_ON_APP);
-      break;
-
-    case TYPE_TABBED:
-    case TYPE_APP_PANEL:
+      delegate_->OnBrowserShortcutClicked(event.flags());
       break;
   }
 
-  // If the item is already active we show a menu - otherwise we activate
-  // the item dependent on its type.
-  // Note that the old launcher has no menu and falls back automatically to
-  // the click action.
-  bool call_object_handler = model_->items()[view_index].type == TYPE_APP_LIST;
-  if (!call_object_handler) {
-    call_object_handler =
-        model_->items()[view_index].status != ash::STATUS_ACTIVE;
-    if (!call_object_handler) {
-      // ShowListMenuForView only returns true if the menu was shown.
-      if (ShowListMenuForView(model_->items()[view_index],
-                              sender)) {
-        // When the menu was shown it is possible that this got deleted.
-        return;
-      }
-      call_object_handler = true;
+  if (event.IsShiftDown())
+    ui::LayerAnimator::set_slow_animation_mode(false);
+
+  if (model_->items()[view_index].type != TYPE_APP_LIST) {
+    // ShowListMenuForView only returns true if the menu was shown.
+    if (ShowListMenuForView(model_->items()[view_index],
+                            sender)) {
+      // When the menu was shown it is possible that this got deleted.
+      return;
     }
   }
-
-  if (call_object_handler) {
-    if (event.IsShiftDown())
-      ui::LayerAnimator::set_slow_animation_mode(true);
-    // The menu was not shown and the objects click handler should be called.
-    switch (model_->items()[view_index].type) {
-      case TYPE_TABBED:
-      case TYPE_APP_PANEL:
-      case TYPE_APP_SHORTCUT:
-      case TYPE_PLATFORM_APP:
-        delegate_->ItemClicked(model_->items()[view_index], event.flags());
-        break;
-      case TYPE_APP_LIST:
-        Shell::GetInstance()->ToggleAppList(GetWidget()->GetNativeView());
-        break;
-      case TYPE_BROWSER_SHORTCUT:
-        delegate_->OnBrowserShortcutClicked(event.flags());
-        break;
-    }
-    if (event.IsShiftDown())
-      ui::LayerAnimator::set_slow_animation_mode(false);
-  }
-
 }
 
 bool LauncherView::ShowListMenuForView(const LauncherItem& item,
                                        views::View* source) {
   scoped_ptr<ui::MenuModel> menu_model;
   menu_model.reset(delegate_->CreateApplicationMenu(item));
 
-  // Make sure we have a menu and it has at least one item in addition to the
-  // application title.
-  if (!menu_model.get() || menu_model->GetItemCount() <= 1)
+  // Make sure we have a menu and it has at least two items in addition to the
+  // application title and the 2 spacing separators.
+  if (!menu_model.get() || menu_model->GetItemCount() <= 4)
     return false;
 
   ShowMenu(menu_model.get(), source, gfx::Point(), false);
   return true;
 }
 
 void LauncherView::ShowContextMenuForView(views::View* source,
                                           const gfx::Point& point) {
   int view_index = view_model_->GetIndexOfView(source);
   if (view_index != -1 &&
@@ skipping 73 matching lines @@
   FOR_EACH_OBSERVER(LauncherIconObserver, observers_,
                     OnLauncherIconPositionsChanged());
   PreferredSizeChanged();
 }
 
 void LauncherView::OnBoundsAnimatorDone(views::BoundsAnimator* animator) {
 }
 
 }  // namespace internal
 }  // namespace ash