Support minimizing the panel into 3-pixel line on Windows. Also support bringing up/down the titl...

chrome/browser/ui/panels/panel_manager.cc

Index: chrome/browser/ui/panels/panel_manager.cc
===================================================================
--- chrome/browser/ui/panels/panel_manager.cc	(revision 90982)
+++ chrome/browser/ui/panels/panel_manager.cc	(working copy)
@@ -9,7 +9,6 @@
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/window_sizer.h"
 
 namespace {
@@ -270,6 +269,29 @@
   DelayedRemove();
 }
 
+bool PanelManager::ShouldBringUpTitleBarForAllMinimizedPanels(

[jennb, 2011/06/29 23:17:54]

Rather than ask every panel if it should bring up title bar, should panel
manager only ask the panel containing the mouse_x point? Or, this could be
platform specific logic. Perhaps what you really want is a way to iterate
through all active panels using PanelManager API?

[jianli, 2011/06/30 01:28:33]

I've thought about providing the iteration API but I rather not give it since I
do not want a panel to talk with any other panel directly.

I also rather leave to the native panel to decide how to check mouse_x and
mouse_y in since the individual platform might have special logic.

+    int mouse_x, int mouse_y) const {
+  for (ActivePanels::const_iterator iter = active_panels_.begin();
+       iter != active_panels_.end(); ++iter) {
+    if ((*iter)->ShouldBringUpTitleBar(mouse_x, mouse_y))
+      return true;
+  }
+  return false;
+}
+
+void PanelManager::BringUpOrDownTitleBarForAllMinimizedPanels(bool bring_up) {
+  for (ActivePanels::const_iterator iter = active_panels_.begin();
+       iter != active_panels_.end(); ++iter) {
+    if (bring_up) {
+      if ((*iter)->expansion_state() == Panel::MINIMIZED)
+        (*iter)->SetExpansionState(Panel::TITLE_ONLY);
+    } else {
+      if ((*iter)->expansion_state() == Panel::TITLE_ONLY)
+        (*iter)->SetExpansionState(Panel::MINIMIZED);
+    }
+  }
+}
+
 void PanelManager::Rearrange(ActivePanels::iterator iter_to_start) {
   if (iter_to_start == active_panels_.end())
     return;
@@ -318,20 +340,6 @@
   return true;
 }
 
-void PanelManager::MinimizeAll() {
-  for (ActivePanels::const_iterator iter = active_panels_.begin();
-       iter != active_panels_.end(); ++iter) {
-    (*iter)->Minimize();
-  }
-}
-
-void PanelManager::RestoreAll() {
-  for (ActivePanels::const_iterator iter = active_panels_.begin();
-       iter != active_panels_.end(); ++iter) {
-    (*iter)->Restore();
-  }
-}
-
 void PanelManager::RemoveAllActive() {
   // This should not be called when we're in the process of dragging.
   DCHECK(dragging_panel_index_ == kInvalidPanelIndex);