Changes tab overview to create a window the max size it can possibly...

chrome/browser/views/tabs/tab_overview_controller.h

 // Copyright (c) 2009 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_VIEWS_TABS_TAB_OVERVIEW_CONTROLLER_H_
 #define CHROME_BROWSER_VIEWS_TABS_TAB_OVERVIEW_CONTROLLER_H_
 
 #include "base/gfx/rect.h"
 #include "base/scoped_ptr.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 
 class Animation;
 class Browser;
 class TabOverviewCell;
 class TabOverviewContainer;
 class TabOverviewGrid;
 
 namespace views {
 class Widget;
 }
 
 // TabOverviewController is responsible for showing a TabOverviewGrid and
 // keeping it in sync with the TabStripModel of a browser.
+//
+// As tabs are added/removed from the TabStripModel the size and position
+// of the container animates. Ideally this would be done by changing the
+// bounds of the host window, but that proved janktastic. Instead the
+// size of the host window is created at the largest possible size the
+// window can be and the bounds of the container are changed during the
+// animation.
 class TabOverviewController : public TabStripModelObserver {
  public:
   // Creates a TabOverviewController that will be shown on the monitor
   // containing |monitor_origin|.
   explicit TabOverviewController(const gfx::Point& monitor_origin);
   ~TabOverviewController();
 
   // Sets the browser we're showing the tab strip for. |horizontal_center|
   // gives the center of the window.
   void SetBrowser(Browser* browser, int horizontal_center);
@@ skipping 44 matching lines @@
  private:
   // Configures a cell from the model.
   void ConfigureCell(TabOverviewCell* cell, int index);
 
   // Removes all the cells in the grid and populates it from the model.
   void RecreateCells();
 
   // Updates the target and start bounds.
   void UpdateStartAndTargetBounds();
 
-  // Sets the bounds of the hosting window to |bounds|.
-  void SetHostBounds(const gfx::Rect& bounds);
+  // Returns the bounds for the tab overview container based on the preferred
+  // size of the container. The returned value is the coordinates of the
+  // root view (container's parent).
+  // See comment above class description for more details.
+  gfx::Rect CalculateContainerBounds();
 
-  // Returns the bounds for the window based on the current content.
+  // Returns the bounds needed for the host.
+  // See comment above class description for more details.
   gfx::Rect CalculateHostBounds();
 
   // The widget showing the view.
   views::Widget* host_;
 
   // Bounds of the monitor we're being displayed on. This is used to position
   // the widget.
   gfx::Rect monitor_bounds_;
 
   // View containing the grid, owned by host.
@@ skipping 28 matching lines @@
   gfx::Rect target_bounds_;
 
   // Are we in the process of mutating the grid? This is used to avoid changing
   // bounds when we're responsible for the mutation.
   bool mutating_grid_;
 
   DISALLOW_COPY_AND_ASSIGN(TabOverviewController);
 };
 
 #endif  // CHROME_BROWSER_VIEWS_TABS_TAB_OVERVIEW_CONTROLLER_H_