Reland with a fix: Do not activate TrayBubbleView by default

ash/system/tray/system_tray_bubble.cc

Index: ash/system/tray/system_tray_bubble.cc
diff --git a/ash/system/tray/system_tray_bubble.cc b/ash/system/tray/system_tray_bubble.cc
index bf94470ed85f0c75f6784d3d8513fc6c2e1cc0de..bd6393915b5e3ab46662fad1acb509894826eb47 100644
--- a/ash/system/tray/system_tray_bubble.cc
+++ b/ash/system/tray/system_tray_bubble.cc
@@ -84,7 +84,7 @@ SystemTrayBubble::~SystemTrayBubble() {
   DestroyItemViews();
   // Reset the host pointer in bubble_view_ in case its destruction is deferred.
   if (bubble_view_)
-    bubble_view_->reset_delegate();
+    bubble_view_->ResetDelegate();
 }
 
 void SystemTrayBubble::UpdateView(
@@ -183,6 +183,9 @@ void SystemTrayBubble::UpdateView(
       }
     }
   }
+
+  // Update bubble as size might have changed.
+  bubble_view_->UpdateBubble();

[yawano, 2017/06/26 04:02:30]

It is necessary to call here.

TrayBubbleView::UpdateBubble moves focus to the first focusable element if extra
keyboard accessibility is enabled.

SystemTray::ShowItems calls SystemTrayBubble::UpdateView if it reuses existing
view, e.g. opening bluetooth submenu in system tray.

For not-reusing existing view case, we are calling TrayBubbleView::UpdateBubble
in TrayBubbleView::InitializeAndShowBubble.

 }
 
 void SystemTrayBubble::InitView(views::View* anchor,
@@ -210,17 +213,6 @@ void SystemTrayBubble::InitView(views::View* anchor,
   }
 }
 
-void SystemTrayBubble::FocusDefaultIfNeeded() {
-  views::FocusManager* manager = bubble_view_->GetFocusManager();
-  if (!manager || manager->GetFocusedView())
-    return;
-
-  views::View* view =
-      manager->GetNextFocusableView(nullptr, nullptr, false, false);
-  if (view)
-    view->RequestFocus();
-}
-
 void SystemTrayBubble::DestroyItemViews() {
   for (std::vector<ash::SystemTrayItem*>::iterator it = items_.begin();
        it != items_.end(); ++it) {