Fix handling of minimized panel count

chrome/browser/ui/panels/panel_overflow_browsertest.cc

Index: chrome/browser/ui/panels/panel_overflow_browsertest.cc
diff --git a/chrome/browser/ui/panels/panel_overflow_browsertest.cc b/chrome/browser/ui/panels/panel_overflow_browsertest.cc
index 14699bda02cede5e5785735886dbbc513685d19a..0df8ac30b40154d16fdcca761aa2564d6062dd6f 100644
--- a/chrome/browser/ui/panels/panel_overflow_browsertest.cc
+++ b/chrome/browser/ui/panels/panel_overflow_browsertest.cc
@@ -317,6 +317,48 @@ IN_PROC_BROWSER_TEST_F(PanelOverflowBrowserTest, CreateOverflowPanels) {
   PanelManager::GetInstance()->CloseAll();
 }
 
+IN_PROC_BROWSER_TEST_F(PanelOverflowBrowserTest, AddMinimizedTillOverflow) {
+  PanelManager* panel_manager = PanelManager::GetInstance();
+  DockedPanelStrip* docked_strip = panel_manager->docked_strip();
+  OverflowPanelStrip* overflow_strip = panel_manager->overflow_strip();
+
+  EXPECT_EQ(0, panel_manager->num_panels());
+  EXPECT_EQ(0, docked_strip->num_panels());
+  EXPECT_EQ(0, overflow_strip->num_panels());
+  EXPECT_FALSE(overflow_strip->overflow_indicator());

[jianli, 2012/03/12 18:08:56]

nit: this can be removed since we do not care about any change to the overflow
indicator in this test.

+
+  CreatePanelParams params("Test", gfx::Rect(0, 0, 100, 100), SHOW_AS_INACTIVE);
+  params.wait_for_fully_created = false;
+
+  unsigned int num_panels = 0;
+  Panel* last_panel;
+  for (; num_panels < 10; ++num_panels) {

[jennb, 2012/03/12 18:26:27]

nit: suggest making the 10 a defined literal or a var.

+    Panel* panel = last_panel = CreatePanelWithParams(params);

[jianli, 2012/03/12 18:08:56]

nit: seems that one variable |panel| is enough.

+    panel->SetExpansionState(Panel::MINIMIZED);
+
+    panel->panel_strip()->RemovePanel(panel);
+
+    docked_strip->AddPanel(panel);

[jianli, 2012/03/12 18:08:56]

Should we call PanelManager::MovePanelToStrip?

+    EXPECT_EQ(panel->expansion_state(), Panel::MINIMIZED);

[jennb, 2012/03/12 18:26:27]

reverse order. it's EXPECT_EQ(expected_value, actual_value).

+  }
+
+  EXPECT_EQ(num_panels, (unsigned int)10);

[jianli, 2012/03/12 18:08:56]

nit: 10u

+  EXPECT_EQ(num_panels, (unsigned int)

[jianli, 2012/03/12 18:08:56]

nit: better to use static_cast

+      (docked_strip->minimized_panel_count() +
+       docked_strip->num_temporary_layout_panels() +
+            overflow_strip->num_panels()));
+
+  WaitForLayoutModeChanged(last_panel, PanelStrip::IN_OVERFLOW);

[jianli, 2012/03/12 18:08:56]

Should we also check if docked_strip->num_temporary_layout_panels() is 0?

[jennb, 2012/03/12 18:26:27]

When you do RemovePanel/AddPanel, the AddPanel will ensure the panel fits in the
dock. I'm surprised the last_panel still moves to overflow...

+
+  EXPECT_EQ(num_panels, (unsigned int)
+            (docked_strip->minimized_panel_count() +
+             docked_strip->num_temporary_layout_panels() +
+             overflow_strip->num_panels()));
+
+  PanelManager::GetInstance()->CloseAll();
+}
+
+
 IN_PROC_BROWSER_TEST_F(PanelOverflowBrowserTest,
                        MAYBE_CreateMoreOverflowPanels) {
   PanelManager* panel_manager = PanelManager::GetInstance();