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

ui/views/focus/focus_manager_unittest.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 <stddef.h>
 
 #include <utility>
 #include <vector>
@@ skipping 860 matching lines @@
   params.delegate = delegate.get();
   child_widget.Init(params);
   View* view1 = new View;
   view1->SetFocusBehavior(View::FocusBehavior::ALWAYS);
   view1->SetBounds(0, 0, 20, 20);
   View* view2 = new View;
   view2->SetFocusBehavior(View::FocusBehavior::ALWAYS);
   view2->SetBounds(20, 0, 20, 20);
   child_widget.client_view()->AddChildView(view1);
   child_widget.client_view()->AddChildView(view2);
-  child_widget.Show();
+  GetWidget()->Show();
   view2->RequestFocus();
   EXPECT_EQ(view2, GetFocusManager()->GetFocusedView());
 
   // Advance focus backwards, which should focus the first.
   GetFocusManager()->AdvanceFocus(false);
   EXPECT_EQ(view1, GetFocusManager()->GetFocusedView());
 
   // Focus forward to |view2|.
   GetFocusManager()->AdvanceFocus(true);
   EXPECT_EQ(view2, GetFocusManager()->GetFocusedView());
 
   // And forward again, wrapping back to |view1|.
   GetFocusManager()->AdvanceFocus(true);
   EXPECT_EQ(view1, GetFocusManager()->GetFocusedView());
 
   // Allow focus to go to the parent, and focus backwards which should now move
   // up |widget_view| (in the parent).
   delegate->set_should_advance_focus_to_parent(true);
   GetFocusManager()->AdvanceFocus(true);
   EXPECT_EQ(widget_view, GetFocusManager()->GetFocusedView());
 }
 
 }  // namespace views