Do not activate system tray bubble by default

ash/common/system/date/date_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/common/system/date/date_view.h"
 
 #include "ash/common/system/tray/system_tray_delegate.h"
 #include "ash/common/system/tray/tray_constants.h"
 #include "ash/common/system/tray/tray_utils.h"
 #include "ash/common/wm_shell.h"
@@ skipping 71 matching lines @@
   UpdateTextInternal(now);
   SchedulePaint();
   SetTimer(now);
 }
 
 void BaseDateTimeView::GetAccessibleState(ui::AXViewState* state) {
   ActionableView::GetAccessibleState(state);
   state->role = ui::AX_ROLE_TIME;
 }
 
-BaseDateTimeView::BaseDateTimeView()
-    : hour_type_(WmShell::Get()->system_tray_delegate()->GetHourClockType()) {
+BaseDateTimeView::BaseDateTimeView(SystemTrayItem* owner)
+    : ActionableView(owner),
+      hour_type_(WmShell::Get()->system_tray_delegate()->GetHourClockType()) {
   SetTimer(base::Time::Now());
   SetFocusBehavior(FocusBehavior::NEVER);
 }
 
 void BaseDateTimeView::SetTimer(const base::Time& now) {
   // Try to set the timer to go off at the next change of the minute. We don't
   // want to have the timer go off more than necessary since that will cause
   // the CPU to wake up and consume power.
   base::Time::Exploded exploded;
   now.LocalExplode(&exploded);
@@ skipping 25 matching lines @@
 void BaseDateTimeView::ChildPreferredSizeChanged(views::View* child) {
   PreferredSizeChanged();
 }
 
 void BaseDateTimeView::OnLocaleChanged() {
   UpdateText();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
-DateView::DateView() : action_(TrayDate::NONE) {
+DateView::DateView(SystemTrayItem* owner)
+    : BaseDateTimeView(owner), action_(TrayDate::NONE) {
   SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
   date_label_ = new views::Label();
   date_label_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   date_label_->SetEnabledColor(kHeaderTextColorNormal);
   UpdateTextInternal(base::Time::Now());
   AddChildView(date_label_);
 }
 
 DateView::~DateView() {}
 
@@ skipping 32 matching lines @@
       IDS_ASH_STATUS_TRAY_DATE, FormatDayOfWeek(now), FormatDate(now)));
 }
 
 bool DateView::PerformAction(const ui::Event& event) {
   if (action_ == TrayDate::NONE)
     return false;
   if (action_ == TrayDate::SHOW_DATE_SETTINGS)
     WmShell::Get()->system_tray_delegate()->ShowDateSettings();
   else if (action_ == TrayDate::SET_SYSTEM_TIME)
     WmShell::Get()->system_tray_delegate()->ShowSetTimeDialog();
+  else
+    return false;
+  CloseSystemBubble();
   return true;
 }
 
 void DateView::OnMouseEntered(const ui::MouseEvent& event) {
   if (action_ == TrayDate::NONE)
     return;
   SetActive(true);
 }
 
 void DateView::OnMouseExited(const ui::MouseEvent& event) {
   if (action_ == TrayDate::NONE)
     return;
   SetActive(false);
 }
 
 void DateView::OnGestureEvent(ui::GestureEvent* event) {
   if (event->type() == ui::ET_GESTURE_TAP_DOWN) {
     SetActive(true);
   } else if (event->type() == ui::ET_GESTURE_TAP_CANCEL ||
              event->type() == ui::ET_GESTURE_END) {
     SetActive(false);
   }
   BaseDateTimeView::OnGestureEvent(event);
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
-TimeView::TimeView(TrayDate::ClockLayout clock_layout) {
+TimeView::TimeView(TrayDate::ClockLayout clock_layout)
+    : BaseDateTimeView(nullptr) {
   SetupLabels();
   UpdateTextInternal(base::Time::Now());
   UpdateClockLayout(clock_layout);
 }
 
 TimeView::~TimeView() {}
 
 void TimeView::UpdateTimeFormat() {
   hour_type_ = WmShell::Get()->system_tray_delegate()->GetHourClockType();
   UpdateText();
@@ skipping 93 matching lines @@
 }
 
 void TimeView::SetupLabel(views::Label* label) {
   label->set_owned_by_client();
   SetupLabelForTray(label);
   label->SetElideBehavior(gfx::NO_ELIDE);
 }
 
 }  // namespace tray
 }  // namespace ash