ash: Move "CAPS LOCK is on." bubble to the notification center.

ash/system/tray_caps_lock.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/system/tray_caps_lock.h"
 
 #include "ash/accessibility_delegate.h"
 #include "ash/resources/grit/ash_resources.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "ash/shell.h"
 #include "ash/shell_port.h"
 #include "ash/strings/grit/ash_strings.h"
+#include "ash/system/system_notifier.h"
 #include "ash/system/tray/actionable_view.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/system/tray/tray_constants.h"
 #include "ash/system/tray/tray_popup_item_style.h"
 #include "ash/system/tray/tray_popup_utils.h"
 #include "ash/system/tray/tri_view.h"
 #include "base/sys_info.h"
 #include "ui/accessibility/ax_node_data.h"
 #include "ui/base/ime/chromeos/ime_keyboard.h"
 #include "ui/base/ime/chromeos/input_method_manager.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/paint_vector_icon.h"
+#include "ui/message_center/message_center.h"
+#include "ui/message_center/notification.h"
 #include "ui/views/border.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/widget/widget.h"
 
+using message_center::Notification;
+
 namespace ash {
 namespace {
 
 // Padding used to position the caption in the caps lock default view row.
 const int kCaptionRightPadding = 6;
 
+const char kCapsLockNotificationId[] = "capslock";
+
 bool CapsLockIsEnabled() {
   chromeos::input_method::InputMethodManager* ime =
       chromeos::input_method::InputMethodManager::Get();
   return (ime && ime->GetImeKeyboard())
              ? ime->GetImeKeyboard()->CapsLockIsEnabled()
              : false;
 }
+
+std::unique_ptr<Notification> CreateNotification() {
+  ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
+  TrayPopupItemStyle default_view_style(
+      TrayPopupItemStyle::FontStyle::DEFAULT_VIEW_LABEL);

[tdanderson, 2017/04/13 17:01:37]

Thanks for making use of TrayPopupItemStyle. However since you are not applying
this style to any labels, I don't think you need to instantiate
|default_view_style|. Instead for the color parameter on line 67, just say:

TrayPopupItemStyle::GetIconColor(TrayPopupItemStyle::ColorStyle::ACTIVE)

[tetsui2, 2017/04/14 01:54:53]

Done.

+  const int string_id =
+      Shell::Get()->system_tray_delegate()->IsSearchKeyMappedToCapsLock()
+          ? IDS_ASH_STATUS_TRAY_CAPS_LOCK_CANCEL_BY_SEARCH
+          : IDS_ASH_STATUS_TRAY_CAPS_LOCK_CANCEL_BY_ALT_SEARCH;
+  std::unique_ptr<Notification> notification(new Notification(
+      message_center::NOTIFICATION_TYPE_SIMPLE, kCapsLockNotificationId,
+      base::string16(), bundle.GetLocalizedString(string_id),
+      gfx::Image(gfx::CreateVectorIcon(kSystemMenuCapsLockIcon,
+                                       default_view_style.GetIconColor())),
+      base::string16() /* display_source */, GURL(),
+      message_center::NotifierId(message_center::NotifierId::SYSTEM_COMPONENT,
+                                 system_notifier::kNotifierCapsLock),
+      message_center::RichNotificationData(), NULL));

[tdanderson, 2017/04/13 17:01:37]

nit: nullptr

[tetsui2, 2017/04/14 01:54:53]

Done.

+  return notification;
 }
 
+}  // namespace
+
 class CapsLockDefaultView : public ActionableView {
  public:
   CapsLockDefaultView()
       : ActionableView(nullptr, TrayPopupInkDropStyle::FILL_BOUNDS),
         text_label_(TrayPopupUtils::CreateDefaultLabel()),
         shortcut_label_(TrayPopupUtils::CreateDefaultLabel()) {
     shortcut_label_->SetEnabled(false);
 
     TriView* tri_view(TrayPopupUtils::CreateDefaultRowView());
     SetLayoutManager(new views::FillLayout);
@@ skipping 73 matching lines @@
 
   // It indicates the shortcut can be used to turn on or turn off Caps Lock.
   views::Label* shortcut_label_;
 
   DISALLOW_COPY_AND_ASSIGN(CapsLockDefaultView);
 };
 
 TrayCapsLock::TrayCapsLock(SystemTray* system_tray)
     : TrayImageItem(system_tray, kSystemTrayCapsLockIcon, UMA_CAPS_LOCK),
       default_(nullptr),
-      detailed_(nullptr),
+      // detailed_(nullptr),

[tdanderson, 2017/04/13 17:01:37]

I think you meant to delete this :)

[tetsui2, 2017/04/14 01:54:53]

Done.

       caps_lock_enabled_(CapsLockIsEnabled()),
       message_shown_(false) {
   chromeos::input_method::InputMethodManager* ime =
       chromeos::input_method::InputMethodManager::Get();
   if (ime && ime->GetImeKeyboard())
     ime->GetImeKeyboard()->AddObserver(this);
 }
 
 TrayCapsLock::~TrayCapsLock() {
   chromeos::input_method::InputMethodManager* ime =
       chromeos::input_method::InputMethodManager::Get();
   if (ime && ime->GetImeKeyboard())
     ime->GetImeKeyboard()->RemoveObserver(this);
 }
 
 void TrayCapsLock::OnCapsLockChanged(bool enabled) {
   caps_lock_enabled_ = enabled;
 
   // Send an a11y alert.
   Shell::Get()->accessibility_delegate()->TriggerAccessibilityAlert(
       enabled ? A11Y_ALERT_CAPS_ON : A11Y_ALERT_CAPS_OFF);
 
   if (tray_view())
     tray_view()->SetVisible(caps_lock_enabled_);
 
   if (default_) {
     default_->Update(caps_lock_enabled_);
   } else {
+    message_center::MessageCenter* message_center =
+        message_center::MessageCenter::Get();
     if (caps_lock_enabled_) {
       if (!message_shown_) {
         ShellPort::Get()->RecordUserMetricsAction(
             UMA_STATUS_AREA_CAPS_LOCK_POPUP);
-        ShowDetailedView(kTrayPopupAutoCloseDelayForTextInSeconds, false);
+
+        message_center->AddNotification(CreateNotification());
         message_shown_ = true;
       }
-    } else if (detailed_) {
-      detailed_->GetWidget()->Close();
+    } else {

[tdanderson, 2017/04/13 17:01:37]

nit: can you just make this an 'else if' here?

[tetsui2, 2017/04/14 01:54:52]

Done.

+      if (message_center->FindVisibleNotificationById(
+              kCapsLockNotificationId)) {
+        message_center->RemoveNotification(kCapsLockNotificationId, false);
+      }
     }
   }
 }
 
 bool TrayCapsLock::GetInitialVisibility() {
   return CapsLockIsEnabled();
 }
 
 views::View* TrayCapsLock::CreateDefaultView(LoginStatus status) {
   if (!caps_lock_enabled_)
     return nullptr;
   DCHECK(!default_);
   default_ = new CapsLockDefaultView;
   default_->Update(caps_lock_enabled_);
   return default_;
 }
 
-views::View* TrayCapsLock::CreateDetailedView(LoginStatus status) {
-  DCHECK(!detailed_);
-  detailed_ = new views::View;
-
-  detailed_->SetLayoutManager(new views::BoxLayout(
-      views::BoxLayout::kHorizontal, kTrayPopupPaddingHorizontal, 10,
-      kTrayPopupPaddingBetweenItems));
-
-  ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
-  views::ImageView* image = new views::ImageView;
-  image->SetImage(
-      CreateVectorIcon(kSystemMenuCapsLockIcon, kMenuIconSize, kMenuIconColor));
-  detailed_->AddChildView(image);
-
-  const int string_id =
-      Shell::Get()->system_tray_delegate()->IsSearchKeyMappedToCapsLock()
-          ? IDS_ASH_STATUS_TRAY_CAPS_LOCK_CANCEL_BY_SEARCH
-          : IDS_ASH_STATUS_TRAY_CAPS_LOCK_CANCEL_BY_ALT_SEARCH;
-  views::Label* label = TrayPopupUtils::CreateDefaultLabel();
-  label->SetText(bundle.GetLocalizedString(string_id));
-  label->SetMultiLine(true);
-  label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
-  detailed_->AddChildView(label);
-  ShellPort::Get()->RecordUserMetricsAction(UMA_STATUS_AREA_CAPS_LOCK_DETAILED);
-
-  return detailed_;
-}
-
 void TrayCapsLock::DestroyDefaultView() {
   default_ = nullptr;
 }
 
-void TrayCapsLock::DestroyDetailedView() {
-  detailed_ = nullptr;
-}
-
 }  // namespace ash