Give up focus if the focused view becomes unfocusable

ui/views/view_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 <map>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/rand_util.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 2752 matching lines @@
 
   View* child = new View();
   root.AddChildView(child);
 
   View* foo1 = new View();
   child->AddChildView(foo1);
   View* foo2 = new View();
   child->AddChildView(foo2);
   View* foo3 = new View();
   child->AddChildView(foo3);
-  foo1->set_focusable(true);
-  foo2->set_focusable(true);
-  foo3->set_focusable(true);
+  foo1->SetFocusable(true);
+  foo2->SetFocusable(true);
+  foo3->SetFocusable(true);
 
   ASSERT_EQ(0, child->GetIndexOf(foo1));
   ASSERT_EQ(1, child->GetIndexOf(foo2));
   ASSERT_EQ(2, child->GetIndexOf(foo3));
   ASSERT_EQ(foo2, foo1->GetNextFocusableView());
   ASSERT_EQ(foo3, foo2->GetNextFocusableView());
   ASSERT_EQ(NULL, foo3->GetNextFocusableView());
 
   // Move |foo2| at the end.
   child->ReorderChildView(foo2, -1);
@@ skipping 104 matching lines @@
   // Check that calling |AddChildView()| does not change the order.
   v1.AddChildView(&v2);
   EXPECT_EQ(0, v1.GetIndexOf(&v2));
   EXPECT_EQ(1, v1.GetIndexOf(&v3));
   v1.AddChildView(&v3);
   EXPECT_EQ(0, v1.GetIndexOf(&v2));
   EXPECT_EQ(1, v1.GetIndexOf(&v3));
 }
 
 ////////////////////////////////////////////////////////////////////////////////
+// FocusManager
+////////////////////////////////////////////////////////////////////////////////
+
+TEST_F(ViewTest, ReviseFocusedViewForUnfocusableView) {
+  // Create a View and focus it.
+  View* view = new View();
+  view->SetFocusable(true);
+
+  scoped_ptr<Widget> widget(new Widget());
+  Widget::InitParams params = CreateParams(Widget::InitParams::TYPE_POPUP);
+  params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  widget->Init(params);
+  widget->GetRootView()->AddChildView(view);
+
+  FocusManager* focus_manager = widget->GetFocusManager();
+  ASSERT_TRUE(focus_manager);
+
+  focus_manager->SetFocusedView(view);
+  EXPECT_EQ(view, focus_manager->GetFocusedView());
+
+  // Disable the focused view and check if it loses focus.
+  view->SetEnabled(false);
+  EXPECT_EQ(NULL, focus_manager->GetFocusedView());
+
+  // Re-enable and re-focus.
+  view->SetEnabled(true);
+  focus_manager->SetFocusedView(view);
+  EXPECT_EQ(view, focus_manager->GetFocusedView());
+
+  // Hide the focused view and check it it loses focus.
+  view->SetVisible(false);
+  EXPECT_EQ(NULL, focus_manager->GetFocusedView());
+
+  // Re-show and re-focus.
+  view->SetVisible(true);
+  focus_manager->SetFocusedView(view);
+  EXPECT_EQ(view, focus_manager->GetFocusedView());
+
+  // Set as not focusable and check if it loses focus.
+  view->SetFocusable(false);
+  EXPECT_EQ(NULL, focus_manager->GetFocusedView());
+}
+
+TEST_F(ViewTest, DisableViewDoesNotActivateWidget) {

[sky, 2013/12/13 17:41:57]

Focus related tests need to be in interactive_ui_tests.

+  // Create first widget and view, activate the widget, and focus the view.
+  scoped_ptr<Widget> widget1(new Widget());
+  Widget::InitParams params1 = CreateParams(Widget::InitParams::TYPE_POPUP);
+  params1.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  widget1->Init(params1);
+
+  View* view1 = new View();
+  view1->SetFocusable(true);
+  widget1->GetRootView()->AddChildView(view1);
+
+  widget1->Activate();
+  EXPECT_TRUE(widget1->IsActive());
+
+  FocusManager* focus_manager1 = widget1->GetFocusManager();
+  ASSERT_TRUE(focus_manager1);
+  focus_manager1->SetFocusedView(view1);
+  EXPECT_EQ(view1, focus_manager1->GetFocusedView());
+
+  // Create second widget and view, activate the widget, and focus the view.
+  scoped_ptr<Widget> widget2(new Widget());
+  Widget::InitParams params2 = CreateParams(Widget::InitParams::TYPE_POPUP);
+  params2.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  widget2->Init(params2);
+
+  View* view2 = new View();
+  view2->SetFocusable(true);
+  widget2->GetRootView()->AddChildView(view2);
+
+  widget2->Activate();
+  EXPECT_TRUE(widget2->IsActive());
+  EXPECT_FALSE(widget1->IsActive());
+
+  FocusManager* focus_manager2 = widget2->GetFocusManager();
+  ASSERT_TRUE(focus_manager2);
+  focus_manager2->SetFocusedView(view2);
+  EXPECT_EQ(view2, focus_manager2->GetFocusedView());
+
+  // Disable the first view and make sure it loses focus, but its widget is not
+  // activated.
+  view1->SetEnabled(false);
+  EXPECT_NE(view1, focus_manager1->GetFocusedView());
+  EXPECT_FALSE(widget1->IsActive());
+  EXPECT_TRUE(widget2->IsActive());
+}
+
+////////////////////////////////////////////////////////////////////////////////
 // Layers
 ////////////////////////////////////////////////////////////////////////////////
 
 #if defined(USE_AURA)
 
 namespace {
 
 // Test implementation of LayerAnimator.
 class TestLayerAnimator : public ui::LayerAnimator {
  public:
@@ skipping 572 matching lines @@
   const std::vector<ui::Layer*>& child_layers_post = root_layer->children();
   ASSERT_EQ(3u, child_layers_post.size());
   EXPECT_EQ(v1->layer(), child_layers_post[0]);
   EXPECT_EQ(v2->layer(), child_layers_post[1]);
   EXPECT_EQ(v1_old_layer, child_layers_post[2]);
 }
 
 #endif  // USE_AURA
 
 }  // namespace views