Remove View::focusable() - Alternate approach.

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 "ui/views/view.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <memory>
@@ skipping 221 matching lines @@
 
   // Exposed as public for testing.
   void DoFocus() {
     views::View::Focus();
   }
 
   void DoBlur() {
     views::View::Blur();
   }
 
-  bool focusable() const { return View::focusable(); }
-
   void set_can_process_events_within_subtree(bool can_process) {
     can_process_events_within_subtree_ = can_process;
   }
 
   bool CanProcessEventsWithinSubtree() const override {
     return can_process_events_within_subtree_;
   }
 
   void Layout() override {
     did_layout_ = true;
@@ skipping 4100 matching lines @@
 
   // Setting parent view should update the child view's layer's offset.
   v1->SetBoundsRect(gfx::Rect(2, 2, 10, 10));
   EXPECT_EQ("0.33 0.33", ToString(v11->layer()->subpixel_position_offset()));
 
   // Setting integral DSF should reset the offset.
   GetRootLayer()->GetCompositor()->SetScaleAndSize(2.0f, size);
   EXPECT_EQ("0.00 0.00", ToString(v11->layer()->subpixel_position_offset()));
 }
 
-TEST_F(ViewTest, FocusableAssertions) {
-  // View subclasses may change insets based on whether they are focusable,
-  // which effects the preferred size. To avoid preferred size changing around
-  // these Views need to key off the last value set to SetFocusBehavior(), not
-  // whether the View is focusable right now. For this reason it's important
-  // that the return value of focusable() depends on the last value passed to
-  // SetFocusBehavior and not whether the View is focusable right now.
-  TestView view;
-  view.SetFocusBehavior(View::FocusBehavior::ALWAYS);
-  EXPECT_TRUE(view.focusable());
-  view.SetEnabled(false);
-  EXPECT_TRUE(view.focusable());
-  view.SetFocusBehavior(View::FocusBehavior::NEVER);
-  EXPECT_FALSE(view.focusable());
-}
-
 // Verifies when a view is deleted it is removed from ViewStorage.
 TEST_F(ViewTest, UpdateViewStorageOnDelete) {
   ViewStorage* view_storage = ViewStorage::GetInstance();
   const int storage_id = view_storage->CreateStorageID();
   {
     View view;
     view_storage->StoreView(storage_id, &view);
   }
   EXPECT_TRUE(view_storage->RetrieveView(storage_id) == NULL);
 }
@@ skipping 176 matching lines @@
   widget.Init(params);
 
   AddViewWithChildLayer(widget.GetRootView());
   ViewThatAddsViewInOnNativeThemeChanged* v =
       new ViewThatAddsViewInOnNativeThemeChanged;
   widget.GetRootView()->AddChildView(v);
   EXPECT_TRUE(v->on_native_theme_changed_called());
 }
 
 }  // namespace views