Update avatar button to MD

chrome/browser/ui/views/frame/glass_browser_frame_view.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_FRAME_GLASS_BROWSER_FRAME_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_GLASS_BROWSER_FRAME_VIEW_H_
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/win/scoped_gdi_object.h"
 #include "chrome/browser/ui/views/frame/avatar_button_manager.h"
 #include "chrome/browser/ui/views/frame/browser_non_client_frame_view.h"
 #include "chrome/browser/ui/views/frame/windows_10_caption_button.h"
 #include "chrome/browser/ui/views/tab_icon_view.h"
 #include "chrome/browser/ui/views/tab_icon_view_model.h"
+#include "chrome/browser/ui/views/tabs/tab_strip.h"
+#include "chrome/browser/ui/views/tabs/tab_strip_observer.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/window/non_client_view.h"
 
 class BrowserView;
 
 class GlassBrowserFrameView : public BrowserNonClientFrameView,
                               public views::ButtonListener,
-                              public TabIconViewModel {
+                              public TabIconViewModel,
+                              public TabStripObserver {
  public:
   // Constructs a non-client view for an BrowserFrame.
   GlassBrowserFrameView(BrowserFrame* frame, BrowserView* browser_view);
   ~GlassBrowserFrameView() override;
 
   // BrowserNonClientFrameView:
   gfx::Rect GetBoundsForTabStrip(views::View* tabstrip) const override;
   int GetTopInset(bool restored) const override;
   int GetThemeBackgroundXInset() const override;
   void UpdateThrobber(bool running) override;
   gfx::Size GetMinimumSize() const override;
-  views::View* GetProfileSwitcherView() const override;
+  views::MenuButton* GetProfileSwitcherButton() const override;
+  void OnBrowserViewInitViewsComplete() override;
 
   // views::NonClientFrameView:
   gfx::Rect GetBoundsForClientView() const override;
   gfx::Rect GetWindowBoundsForClientBounds(
       const gfx::Rect& client_bounds) const override;
   int NonClientHitTest(const gfx::Point& point) override;
   void UpdateWindowIcon() override;
   void UpdateWindowTitle() override;
   void GetWindowMask(const gfx::Size& size, gfx::Path* window_mask) override {}
   void ResetWindowControls() override {}
@@ skipping 20 matching lines @@
   void Layout() override;
 
   // BrowserNonClientFrameView:
   void UpdateProfileIcons() override;
 
  private:
   // views::NonClientFrameView:
   bool DoesIntersectRect(const views::View* target,
                          const gfx::Rect& rect) const override;
 
+  // TabStripObserver:
+  void TabStripMaxXChanged(TabStrip* tab_strip) override;
+  void TabStripDeleted(TabStrip* tab_strip) override;
+  void TabStripRemovedTabAt(TabStrip* tab_strip, int index) override;
+
   // Returns the thickness of the border around the client area (web content,
   // toolbar, and tabs) that separates it from the frame border. If |restored|
   // is true, this is calculated as if the window was restored, regardless of
   // its current state.
   int ClientBorderThickness(bool restored) const;
 
   // Returns the thickness of the window border for the left, right, and bottom
   // edges of the frame. On Windows 10 this is a mostly-transparent handle that
   // allows you to resize the window.
   int FrameBorderThickness() const;
@@ skipping 96 matching lines @@
   Windows10CaptionButton* maximize_button_;
   Windows10CaptionButton* restore_button_;
   Windows10CaptionButton* close_button_;
 
   // Whether or not the window throbber is currently animating.
   bool throbber_running_;
 
   // The index of the current frame of the throbber animation.
   int throbber_frame_;
 
+  // TabStrip, which we need to watch for changes to resize the avatar button

[msarda, 2017/04/21 09:43:28]

It is not clear what "we" means in this context:
* is it the developper?
* is it this class?

Please rephrase. Maybe something like: 
// The tab strip, which needs to be observed for changes in order to resize the
avatar button.

[emx, 2017/04/24 16:23:10]

Done.

+  TabStrip* tab_strip_;
+
   static const int kThrobberIconCount = 24;
   static HICON throbber_icons_[kThrobberIconCount];
   static void InitThrobberIcons();
 
   DISALLOW_COPY_AND_ASSIGN(GlassBrowserFrameView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_GLASS_BROWSER_FRAME_VIEW_H_