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

chrome/browser/ui/panels/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_PANEL_H_
 #define CHROME_BROWSER_UI_PANELS_PANEL_H_
 #pragma once
 
 #include "chrome/browser/ui/browser_window.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "ui/gfx/rect.h"
 
 class NativePanel;
 class PanelManager;
 
 // A platform independent implementation of BrowserWindow for Panels.  This
 // class would get the first crack at all the BrowserWindow calls for Panels and
 // do one or more of the following:
 // - Do nothing.  The function is not relevant to Panels.
 // - Throw an exceptions.  The function shouldn't be called for Panels.
 // - Do Panel specific platform independent processing and then invoke the
 //   function on the platform specific BrowserWindow member.  For example,
 //   Panel size is restricted to certain limits.
 // - Invoke an appropriate PanelManager function to do stuff that might affect
 //   other Panels.  For example deleting a panel would rearrange other panels.
 class Panel : public BrowserWindow {
  public:
+  enum ExpansionState {
+   EXPANDED,
+   TITLE_ONLY,
+   MINIMIZED
+  };
+
   virtual ~Panel();
 
   // Returns the PanelManager associated with this panel.
   PanelManager* manager() const;
 
-  void Minimize();
-  void Restore();
+  // Sets the expansion state to one of the following values:
+  // * Fully expanded

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

Move these to comments on the enum values.

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

Done.

+  // * Shown with title-bar
+  // * Minimized to 3-pixel lines
+  void SetExpansionState(ExpansionState expansion_state);
+
+  // Should we bring up the title-bar given the current mouse point?

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

Comments should explain what "bring up title-bar" means. Otherwise, these
comments simply repeat the name of the function without providing additional
info.

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

Done.

+  bool ShouldBringUpTitleBar(int mouse_x, int mouse_y) const;
 
   // BrowserWindow overrides.
   virtual void Show() OVERRIDE;
   virtual void ShowInactive() OVERRIDE;
   virtual void SetBounds(const gfx::Rect& bounds) OVERRIDE;
   virtual void Close() OVERRIDE;
   virtual void Activate() OVERRIDE;
   virtual void Deactivate() OVERRIDE;
   virtual bool IsActive() const OVERRIDE;
   virtual void FlashFrame() OVERRIDE;
@@ skipping 91 matching lines @@
                                         const gfx::Rect& bounds);
 
   // Gets the extension from the browser that a panel is created from.
   // Returns NULL if it cannot be found.
   static const Extension* GetExtension(Browser* browser);
 
 #ifdef UNIT_TEST
   NativePanel* native_panel() { return native_panel_; }
 #endif
 
+  ExpansionState expansion_state() const { return expansion_state_; }
+
  protected:
   virtual void DestroyBrowser() OVERRIDE;
 
  private:
   friend class PanelManager;
 
   // Panel can only be created using PanelManager::CreatePanel().
   Panel(Browser* browser, const gfx::Rect& bounds);
 
   // This is different from BrowserWindow::SetBounds():
   // * SetPanelBounds() is only called by PanelManager to manage its position.
   // * SetBounds() is called by the API to try to change the bounds, which is
   //   not allowed for Panel.
   void SetPanelBounds(const gfx::Rect& bounds);
 
   // Platform specifc implementation for panels.  It'd be one of
   // PanelBrowserWindowGtk/PanelBrowserView/PanelBrowserWindowCocoa.
   NativePanel* native_panel_;  // Weak, owns us.
 
+  ExpansionState expansion_state_;
+
   DISALLOW_COPY_AND_ASSIGN(Panel);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_H_