Panel strip refactor cleanup.

chrome/browser/ui/panels/panel.h

Index: chrome/browser/ui/panels/panel.h
diff --git a/chrome/browser/ui/panels/panel.h b/chrome/browser/ui/panels/panel.h
index c9f4eab76d2a2a153a3d0b44b196089547375016..b72511e6a3735b5c1e9269a2876803bf0323f24c 100644
--- a/chrome/browser/ui/panels/panel.h
+++ b/chrome/browser/ui/panels/panel.h
@@ -58,8 +58,6 @@ class Panel : public BrowserWindow,
 
   void SetExpansionState(ExpansionState new_expansion_state);
 
-  bool ShouldBringUpTitlebar(int mouse_x, int mouse_y) const;
-
   bool IsDrawingAttention() const;
 
   // This function will only get called by PanelManager when full screen mode
@@ -69,8 +67,6 @@ class Panel : public BrowserWindow,
   // b) it remains on top when an app exits full screen mode.
   void FullScreenModeChanged(bool is_full_screen);
 
-  void MoveOutOfOverflow();
-
   // Ensures that the panel is fully visible, that is, not obscured by other
   // top-most windows.
   void EnsureFullyVisible();
@@ -248,6 +244,14 @@ class Panel : public BrowserWindow,
   // Sets minimum and maximum size for the panel.
   void SetSizeRange(const gfx::Size& min_size, const gfx::Size& max_size);
 
+  // Sets whether the panel app icon is visible in the taskbar.
+  void SetAppIconVisibility(bool visible);
+
+  // Newly created panels may be placed in a temporary layout until their
+  // final position is determined.
+  bool temporary_layout() const { return temporary_layout_; }
+  void set_temporary_layout(bool temporary) { temporary_layout_ = temporary; }
+
  protected:
   virtual void DestroyBrowser() OVERRIDE;
 
@@ -272,6 +276,10 @@ class Panel : public BrowserWindow,
 
   bool initialized_;
 
+  // Newly created panels may be placed in a temporary layout until their
+  // final position is determined.
+  bool temporary_layout_;

[jianli, 2011/12/07 22:13:26]

Probably better to say has_temporary_layout_. The name "temporary_layout_" makes
me think this stores the temporary layout info, instead of a boolean flag. Or,
maybe we can call it as layout_finalized_.

[jennb, 2011/12/07 23:35:40]

Done.

+
   // Stores the full size of the panel so we can restore it after it's
   // been minimized.
   gfx::Size restored_size_;
@@ -292,6 +300,9 @@ class Panel : public BrowserWindow,
 
   ExpansionState expansion_state_;
 
+  // Indicates whether the panel app icon is visible in the taskbar.
+  bool app_icon_visible_;
+
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(Panel);