Chromium Code Reviews| Index: chrome/browser/ui/panels/panel.h |
| =================================================================== |
| --- chrome/browser/ui/panels/panel.h (revision 90982) |
| +++ chrome/browser/ui/panels/panel.h (working copy) |
| @@ -26,14 +26,26 @@ |
| // 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"
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; |
| @@ -144,6 +156,8 @@ |
| NativePanel* native_panel() { return native_panel_; } |
| #endif |
| + ExpansionState expansion_state() const { return expansion_state_; } |
| + |
| protected: |
| virtual void DestroyBrowser() OVERRIDE; |
| @@ -163,6 +177,8 @@ |
| // PanelBrowserWindowGtk/PanelBrowserView/PanelBrowserWindowCocoa. |
| NativePanel* native_panel_; // Weak, owns us. |
| + ExpansionState expansion_state_; |
| + |
| DISALLOW_COPY_AND_ASSIGN(Panel); |
| }; |