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

ui/views/widget/root_view_unittest.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 "ui/views/widget/root_view.h"
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "ui/events/event_utils.h"
 #include "ui/views/context_menu_controller.h"
@@ skipping 26 matching lines @@
 };
 
 // Verifies deleting a View in OnKeyPressed() doesn't crash and that the
 // target is marked as destroyed in the returned EventDispatchDetails.
 TEST_F(RootViewTest, DeleteViewDuringKeyEventDispatch) {
   Widget widget;
   Widget::InitParams init_params =
       CreateParams(Widget::InitParams::TYPE_POPUP);
   init_params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   widget.Init(init_params);
+  widget.Show();
 
   bool got_key_event = false;
 
   View* content = new View;
   widget.SetContentsView(content);
 
   View* child = new DeleteOnKeyEventView(&got_key_event);
   content->AddChildView(child);
 
   // Give focus to |child| so that it will be the target of the key event.
@@ skipping 50 matching lines @@
 };
 
 // Tests that context menus are shown for certain key events (Shift+F10
 // and VKEY_APPS) by the pre-target handler installed on RootView.
 TEST_F(RootViewTest, ContextMenuFromKeyEvent) {
   Widget widget;
   Widget::InitParams init_params =
       CreateParams(Widget::InitParams::TYPE_POPUP);
   init_params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   widget.Init(init_params);
+  widget.Show();
   internal::RootView* root_view =
       static_cast<internal::RootView*>(widget.GetRootView());
 
   TestContextMenuController controller;
   View* focused_view = new View;
   focused_view->set_context_menu_controller(&controller);
   widget.SetContentsView(focused_view);
   focused_view->SetFocusBehavior(View::FocusBehavior::ALWAYS);
   focused_view->RequestFocus();
 
@@ skipping 406 matching lines @@
   // mouse exit event to |subchild| and destroy the widget. This should not
   // crash when the mouse exit handler returns from |subchild|.
   ui::MouseEvent move_event2(ui::ET_MOUSE_MOVED, gfx::Point(15, 15),
                              gfx::Point(15, 15), ui::EventTimeForNow(), 0, 0);
   root_view->OnMouseMoved(move_event2);
   EXPECT_FALSE(widget_deletion_observer.IsWidgetAlive());
 }
 
 }  // namespace test
 }  // namespace views