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

ui/views/view_targeter_unittest.cc

 // Copyright 2014 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/view_targeter.h"
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "ui/events/event_targeter.h"
 #include "ui/events/event_utils.h"
@@ skipping 107 matching lines @@
 
 // Verifies that the the functions ViewTargeter::FindTargetForEvent()
 // and ViewTargeter::FindNextBestTarget() are implemented correctly
 // for key events.
 TEST_F(ViewTargeterTest, ViewTargeterForKeyEvents) {
   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();
 
   View* content = new View;
   View* child = new View;
   View* grandchild = new View;
 
   widget.SetContentsView(content);
   content->AddChildView(child);
   child->AddChildView(grandchild);
 
   grandchild->SetFocusBehavior(View::FocusBehavior::ALWAYS);
@@ skipping 527 matching lines @@
   EXPECT_EQ(v1, root_view->GetTooltipHandlerForPoint(v1_origin));
   EXPECT_EQ(root_view, root_view->GetTooltipHandlerForPoint(v2_origin));
 
   EXPECT_FALSE(v1->GetTooltipHandlerForPoint(v2_origin));
 
   widget->CloseNow();
 }
 
 }  // namespace test
 }  // namespace views