Do not give instant focus if a view's toplevelwidget is not active

ui/views/focus/focus_manager.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 "ui/views/focus/focus_manager.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/auto_reset.h"
@@ skipping 295 matching lines @@
   // Disabling keyboard accessibility may cause the focused view to become not
   // focusable. Hence advance focus if necessary.
   AdvanceFocusIfNecessary();
 }
 
 void FocusManager::SetFocusedViewWithReason(
     View* view, FocusChangeReason reason) {
   if (focused_view_ == view)
     return;
 
+#if !defined(OS_MACOSX)
+  // TODO(warx): There are some AccessiblePaneViewTest failed on macosx.
+  // crbug.com/650859. Remove !defined(OS_MACOSX) once that is fixed.
+  //
+  // If |widget_| is not active, focus is not allowed to set within |widget_|

[sky, 2016/10/04 15:47:03]

I think you want something like:

If the widget isn't active store the focused view and then attempt to activate
the widget. If activation succeeds |view| will be focused. If activation fails
|view| will be focused the next time the widget is made active.

[Qiang(Joe) Xu, 2016/10/10 16:40:00]

Done.

+  // right now, we just need to store this view.
+  if (view && !widget_->IsActive()) {
+    SetStoredFocusView(view);
+    widget_->Activate();
+    return;
+  }
+#endif
+
   base::AutoReset<bool> auto_changing_focus(&is_changing_focus_, true);
   // Update the reason for the focus change (since this is checked by
   // some listeners), then notify all listeners.
   focus_change_reason_ = reason;
   FOR_EACH_OBSERVER(FocusChangeListener, focus_change_listeners_,
                     OnWillChangeFocus(focused_view_, view));
 
   View* old_focused_view = focused_view_;
   focused_view_ = view;
   if (old_focused_view)
@@ skipping 227 matching lines @@
 // |keyboard_accessible_| is only used on Mac.
 #if defined(OS_MACOSX)
   return keyboard_accessible_ ? view->IsAccessibilityFocusable()
                               : view->IsFocusable();
 #else
   return view->IsAccessibilityFocusable();
 #endif
 }
 
 }  // namespace views