Chromium Code Reviews| Index: chrome/browser/ui/toolbar/toolbar_actions_bar_unittest.cc |
| diff --git a/chrome/browser/ui/toolbar/toolbar_actions_bar_unittest.cc b/chrome/browser/ui/toolbar/toolbar_actions_bar_unittest.cc |
| index a127432e0dbf96d9c52b44fa883c2a173045ca14..e62c9d2a167b347ac2fcd3850b1c76a5303a31fc 100644 |
| --- a/chrome/browser/ui/toolbar/toolbar_actions_bar_unittest.cc |
| +++ b/chrome/browser/ui/toolbar/toolbar_actions_bar_unittest.cc |
| @@ -224,11 +224,8 @@ TEST_P(ToolbarActionsBarUnitTest, BasicToolbarActionsBarTest) { |
| // Since all icons are showing, the current width should be the max width. |
| int maximum_width = expected_width; |
| EXPECT_EQ(maximum_width, toolbar_actions_bar()->GetMaximumWidth()); |
| - // The minimum width should be just enough for the chevron to be displayed |
| - // along with left and right padding. |
| - int minimum_width = 2 * platform_settings.item_spacing + |
| - toolbar_actions_bar()->delegate_for_test()-> |
| - GetChevronWidth(); |
| + // The minimum width should be just enough some padding. |
|
Evan Stade
2016/05/24 17:58:55
I'm stymied by this sentence
Devlin
2016/05/24 21:59:00
Rephrased to be more clear.
|
| + int minimum_width = platform_settings.item_spacing; |
| EXPECT_EQ(minimum_width, toolbar_actions_bar()->GetMinimumWidth()); |
| // Test the connection between the ToolbarActionsBar and the model by |
| @@ -236,14 +233,12 @@ TEST_P(ToolbarActionsBarUnitTest, BasicToolbarActionsBarTest) { |
| toolbar_model()->SetVisibleIconCount(2u); |
| EXPECT_EQ(2u, toolbar_actions_bar()->GetIconCount()); |
| - // The current width should now be enough for two icons, and the chevron. |
| + // The current width should now be enough for two icons. |
| // IconWidth(true) includes the spacing to the left of each icon and |
| // |item_spacing| accounts for the spacing to the right of the rightmost |
| // icon. |
| expected_width = 2 * ToolbarActionsBar::IconWidth(true) + |
| - platform_settings.item_spacing + |
| - toolbar_actions_bar()->delegate_for_test()-> |
| - GetChevronWidth(); |
| + platform_settings.item_spacing; |
| EXPECT_EQ(expected_width, toolbar_actions_bar()->GetPreferredSize().width()); |
| // The maximum and minimum widths should have remained constant (since we have |
| // the same number of actions). |