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

chrome/browser/ui/panels/native_panel.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_PANELS_NATIVE_PANEL_H_
 #define CHROME_BROWSER_UI_PANELS_NATIVE_PANEL_H_
 #pragma once
 
 #include "ui/gfx/native_widget_types.h"
 
@@ skipping 16 matching lines @@
 class NativePanel {
   friend class Panel;
 
  protected:
   virtual ~NativePanel() {}
 
   virtual void ShowPanel() = 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 RestorePanel(bool titlebar_only) = 0;

[Dmitry Titov, 2011/06/24 02:05:45]

Hmm.

So we had 2 states (minimized and restored) and 2 methods to switch into one of
them. Now we've got 3rd state (show titlebar only).

I would suggest using 3rd method, or perhaps even have one method
(SetFrameMode?) with enum parameter (MINIMIZED, TITLE_ONLY, NORMAL?) rather then
modify the behavior of RestorePanel with a boolean.

[jianli, 2011/06/29 01:28:12]

As discussed, I changed to using one method SetPanelExpansionState. I also
changed to name the enum as ExpansionState.

   virtual void ClosePanel() = 0;
   virtual void ActivatePanel() = 0;
   virtual void DeactivatePanel() = 0;
   virtual bool IsPanelActive() const = 0;
   virtual gfx::NativeWindow GetNativePanelHandle() = 0;
   virtual void UpdatePanelTitleBar() = 0;
   virtual void ShowTaskManagerForPanel() = 0;
   virtual void NotifyPanelOnUserChangedTheme() = 0;
   virtual void FlashPanelFrame() = 0;
   virtual void DestroyPanelBrowser() = 0;
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_NATIVE_PANEL_H_