[Win] Redesign the new avatar button.

chrome/browser/ui/views/tabs/tab_strip.h

 // Copyright (c) 2012 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_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 239 matching lines @@
   virtual int OnDragUpdated(const ui::DropTargetEvent& event) OVERRIDE;
   virtual void OnDragExited() OVERRIDE;
   virtual int OnPerformDrop(const ui::DropTargetEvent& event) OVERRIDE;
   virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
   virtual views::View* GetTooltipHandlerForPoint(
       const gfx::Point& point) OVERRIDE;
 
   // Returns preferred height in immersive style.
   static int GetImmersiveHeight();
 
+  // The size of the new tab button must be hardcoded because we need to be
+  // able to lay it out before we are able to get its image from the
+  // ui::ThemeProvider.  It also makes sense to do this, because the size of the
+  // new tab button should not need to be calculated dynamically.
+  static const int kNewTabButtonAssetWidth;

[sky, 2014/09/26 03:45:16]

constants should be first in a section (see style guide).

[noms (inactive), 2014/09/26 16:03:42]

Done.

+  static const int kNewTabButtonAssetHeight;
+
  private:
   typedef std::vector<Tab*> Tabs;
   typedef std::map<int, Tabs> TabsClosingMap;
   typedef std::pair<TabsClosingMap::iterator,
                     Tabs::iterator> FindClosingTabResult;
 
   class RemoveTabDelegate;
 
   friend class TabDragController;
   friend class TabDragControllerTest;
@@ skipping 33 matching lines @@
     // Whether the MIME type of the file pointed to by |url| is supported.
     bool file_supported;
 
    private:
     DISALLOW_COPY_AND_ASSIGN(DropInfo);
   };
 
   // Horizontal gap between mini and non-mini-tabs.
   static const int kMiniToNonMiniGap;
 
-  // The size of the new tab button must be hardcoded because we need to be
-  // able to lay it out before we are able to get its image from the
-  // ui::ThemeProvider.  It also makes sense to do this, because the size of the
-  // new tab button should not need to be calculated dynamically.
-  static const int kNewTabButtonAssetWidth;
-  static const int kNewTabButtonAssetHeight;
-
   void Init();
 
   // Creates and returns a new tab. The caller owners the returned tab.
   Tab* CreateTab();
 
   // Invoked from |AddTabAt| after the newly created tab has been inserted.
   void StartInsertTabAnimation(int model_index);
 
   // Invoked from |MoveTab| after |tab_data_| has been updated to animate the
   // move.
@@ skipping 348 matching lines @@
   bool immersive_style_;
 
   // Our observers.
   typedef ObserverList<TabStripObserver> TabStripObservers;
   TabStripObservers observers_;
 
   DISALLOW_COPY_AND_ASSIGN(TabStrip);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_TAB_STRIP_H_