Index: chrome/browser/ui/views/toolbar/browser_actions_container_browsertest.cc |
diff --git a/chrome/browser/ui/views/toolbar/browser_actions_container_browsertest.cc b/chrome/browser/ui/views/toolbar/browser_actions_container_browsertest.cc |
index fc611ff71500ec237e6288626c2fef8893240c2f..c0a6233e8e1870ac6d0750413e492dc8a9de6b9e 100644 |
--- a/chrome/browser/ui/views/toolbar/browser_actions_container_browsertest.cc |
+++ b/chrome/browser/ui/views/toolbar/browser_actions_container_browsertest.cc |
@@ -116,7 +116,7 @@ |
EXPECT_EQ(extension_b()->id(), browser_actions_bar()->GetExtensionId(2)); |
EXPECT_EQ(3u, container->VisibleBrowserActions()); |
EXPECT_FALSE(container->chevron()->visible()); |
- EXPECT_TRUE(model->all_icons_visible()); |
+ EXPECT_EQ(-1, model->GetVisibleIconCount()); |
// TODO(devlin): Ideally, we'd also have tests for dragging from the legacy |
// overflow menu (i.e., chevron) to the main bar, but this requires either |
@@ -438,8 +438,7 @@ |
overflow_bar()->Layout(); |
// All actions are showing, and are in the installation order. |
- EXPECT_TRUE(model()->all_icons_visible()); |
- EXPECT_EQ(3u, model()->visible_icon_count()); |
+ EXPECT_EQ(-1, model()->GetVisibleIconCount()); |
ASSERT_EQ(3u, main_bar()->num_toolbar_actions()); |
EXPECT_EQ(extension_a()->id(), main_bar()->GetIdAt(0u)); |
EXPECT_EQ(extension_b()->id(), main_bar()->GetIdAt(1u)); |