| 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));
|
|
|