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

chrome/browser/ui/panels/native_panel.h

Index: chrome/browser/ui/panels/native_panel.h
===================================================================
--- chrome/browser/ui/panels/native_panel.h	(revision 90982)
+++ chrome/browser/ui/panels/native_panel.h	(working copy)
@@ -6,6 +6,7 @@
 #define CHROME_BROWSER_UI_PANELS_NATIVE_PANEL_H_
 #pragma once
 
+#include "chrome/browser/ui/panels/panel.h"

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

Gut reaction was this include felt circular. We're only including it for the
ExpansionState enum, right? Maybe that should go in some panel_constants.h
file...

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

It is really not circular. Though it is kind of ugly, we do not want to move it
to a new header file that has it exposed directly. I keep it as now per our
discussion.

 #include "ui/gfx/native_widget_types.h"
 
 class Panel;
@@ -34,8 +35,9 @@
   virtual void ShowPanelInactive() = 0;
   virtual gfx::Rect GetPanelBounds() const = 0;
   virtual void SetPanelBounds(const gfx::Rect& bounds) = 0;
-  virtual void MinimizePanel() = 0;
-  virtual void RestorePanel() = 0;
+  virtual void SetPanelExpansionState(
+      Panel::ExpansionState expansion_state) = 0;
+  virtual bool ShouldBringUpPanelTitleBar(int mouse_x, int mouse_y) const = 0;

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

Could you add comments explaining what x and y are and describe the purpose of
this method?

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

Done.

   virtual void ClosePanel() = 0;
   virtual void ActivatePanel() = 0;
   virtual void DeactivatePanel() = 0;