When the chrome window is maximized, make sure the new tab button extends

chrome/browser/ui/views/tabs/tab_strip.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_VIEWS_TABS_TAB_STRIP_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_TAB_STRIP_H_
 #pragma once
 
 #include "base/memory/ref_counted.h"
 #include "base/timer.h"
 #include "chrome/browser/ui/views/tabs/base_tab_strip.h"
 #include "ui/base/animation/animation_container.h"
 #include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
 #include "views/controls/button/image_button.h"
 #include "views/mouse_watcher.h"
 
+class Browser;
 class Tab;
 class TabStripSelectionModel;
 
 namespace views {
 class ImageView;
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 //
 // TabStrip
 //
 //  A View that represents the TabStripModel. The TabStrip has the
 //  following responsibilities:
 //    - It implements the TabStripModelObserver interface, and acts as a
 //      container for Tabs, and is also responsible for creating them.
 //    - It takes part in Tab Drag & Drop with Tab, TabDragHelper and
 //      DraggedTab, focusing on tasks that require reshuffling other tabs
 //      in response to dragged tabs.
 //
 ///////////////////////////////////////////////////////////////////////////////
 class TabStrip : public BaseTabStrip,
                  public views::ButtonListener,
                  public views::MouseWatcherListener {
  public:
-  explicit TabStrip(TabStripController* controller);
+  explicit TabStrip(Browser* browser, TabStripController* controller);

[sky, 2011/07/14 19:51:08]

TabStrip shouldn't depend on browser. Move the conditional into
TabStripController and BrowserTabStripController.

   virtual ~TabStrip();
 
-  // Creates the new tab button.
-  void InitTabStripButtons();
-
   // Returns the bounds of the new tab button.
   gfx::Rect GetNewTabButtonBounds();
 
+  // Does he new tab button need to be sized to the top of the tab strip?
+  bool SizeTabButtonToTopOfTabStrip();
+
   // MouseWatcherListener overrides:
   virtual void MouseMovedOutOfView() OVERRIDE;
 
   // AbstractTabStripView implementation:
   virtual bool IsPositionInWindowCaption(const gfx::Point& point) OVERRIDE;
   virtual void SetBackgroundOffset(const gfx::Point& offset) OVERRIDE;
 
   // BaseTabStrip implementation:
   virtual void PrepareForCloseAt(int model_index) OVERRIDE;
   virtual void RemoveTabAt(int model_index) OVERRIDE;
@@ skipping 71 matching lines @@
     // Renders the drop indicator.
     views::Widget* arrow_window;
     views::ImageView* arrow_view;
 
    private:
     DISALLOW_COPY_AND_ASSIGN(DropInfo);
   };
 
   void Init();
 
+  // Creates the new tab button.
+  void InitTabStripButtons();
+
   // Set the images for the new tab button.
   void LoadNewTabButtonImage();
 
   // Retrieves the Tab at the specified index. Remember, the specified index
   // is in terms of tab_data, *not* the model.
   Tab* GetTabAtTabDataIndex(int tab_data_index) const;
 
   // Returns the tab at the specified index. If a remove animation is on going
   // and the index is >= the index of the tab being removed, the index is
   // incremented. While a remove operation is on going the indices of the model
@@ skipping 109 matching lines @@
 
   // To ensure all tabs pulse at the same time they share the same animation
   // container. This is that animation container.
   scoped_refptr<ui::AnimationContainer> animation_container_;
 
   // Used for stage 1 of new tab animation.
   base::OneShotTimer<TabStrip> new_tab_timer_;
 
   scoped_ptr<views::MouseWatcher> mouse_watcher_;
 
+  // Pointer to the browser that holds this tab strip.
+  Browser* browser_;
+
   DISALLOW_COPY_AND_ASSIGN(TabStrip);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_TAB_STRIP_H_