Do not activate system tray bubble by default

ash/common/system/chromeos/tray_tracing.cc

 // Copyright 2013 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/chromeos/tray_tracing.h"
 
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/metrics/user_metrics_action.h"
 #include "ash/common/system/tray/actionable_view.h"
 #include "ash/common/system/tray/fixed_sized_image_view.h"
@@ skipping 10 matching lines @@
 #include "ui/gfx/vector_icons_public.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 
 namespace ash {
 namespace tray {
 
 class DefaultTracingView : public ActionableView {
  public:
-  DefaultTracingView() {
+  DefaultTracingView(SystemTrayItem* owner) : ActionableView(owner) {

[James Cook, 2016/09/22 21:17:16]

explicit

[oshima, 2016/09/23 09:37:19]

Done.

     SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal,
                                           kTrayPopupPaddingHorizontal, 0,
                                           kTrayPopupPaddingBetweenItems));
 
     ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
     image_ =
         new FixedSizedImageView(0, GetTrayConstant(TRAY_POPUP_ITEM_HEIGHT));
     if (MaterialDesignController::UseMaterialDesignSystemIcons()) {
       // TODO(tdanderson): Update the icon used for tracing or remove it from
       // the system menu. See crbug.com/625691.
@@ skipping 13 matching lines @@
   }
 
   ~DefaultTracingView() override {}
 
  private:
   // Overridden from ActionableView.
   bool PerformAction(const ui::Event& event) override {
     WmShell::Get()->RecordUserMetricsAction(
         UMA_STATUS_AREA_TRACING_DEFAULT_SELECTED);
     WmShell::Get()->system_tray_delegate()->ShowChromeSlow();
+    CloseSystemBubble();
     return true;
   }
 
   views::ImageView* image_;
   views::Label* label_;
 
   DISALLOW_COPY_AND_ASSIGN(DefaultTracingView);
 };
 
 }  // namespace tray
@@ skipping 17 matching lines @@
     tray_view()->SetVisible(visible);
 }
 
 bool TrayTracing::GetInitialVisibility() {
   return false;
 }
 
 views::View* TrayTracing::CreateDefaultView(LoginStatus status) {
   CHECK(default_ == NULL);
   if (tray_view() && tray_view()->visible())
-    default_ = new tray::DefaultTracingView();
+    default_ = new tray::DefaultTracingView(this);
   return default_;
 }
 
 views::View* TrayTracing::CreateDetailedView(LoginStatus status) {
   return NULL;
 }
 
 void TrayTracing::DestroyDefaultView() {
   default_ = NULL;
 }
 
 void TrayTracing::DestroyDetailedView() {}
 
 void TrayTracing::OnTracingModeChanged(bool value) {
   SetTrayIconVisible(value);
 }
 
 }  // namespace ash