Update avatar button to MD (part 1)

chrome/browser/ui/views/frame/minimize_button_metrics_win.h

Index: chrome/browser/ui/views/frame/minimize_button_metrics_win.h
diff --git a/chrome/browser/ui/views/frame/minimize_button_metrics_win.h b/chrome/browser/ui/views/frame/minimize_button_metrics_win.h
index 86d08a4b5f9da28c860b3db691d96cb67eabb6a9..3319dc14340ef2bb48b07f0369d284c363ac3090 100644
--- a/chrome/browser/ui/views/frame/minimize_button_metrics_win.h
+++ b/chrome/browser/ui/views/frame/minimize_button_metrics_win.h
@@ -18,7 +18,8 @@ class MinimizeButtonMetrics {
   MinimizeButtonMetrics();
   ~MinimizeButtonMetrics();
 
-  // Returns the height of the native caption buttons in DIPs.
+  // Return the width/height of the native caption buttons in DIPs.

[Peter Kasting, 2017/05/11 01:03:43]

Nit: "Returns" was correct; see bit about "descriptive vs. imperative" in
http://google.github.io/styleguide/cppguide.html#Function_Comments .

[emx, 2017/05/11 12:56:11]

Oh, it was supposed to be plural to apply to both functions. :) But I've
reverted it to one function.

+  static int GetCaptionButtonWidthInDIPs();
   static int GetCaptionButtonHeightInDIPs();

[Peter Kasting, 2017/05/11 01:03:43]

Nit: Maybe

  static gfx::Size GetCaptionButtonSize();

?  (DIPs are implied with a gfx::Size, at least today.)

[emx, 2017/05/11 12:56:11]

Good point, I'll use this if we ever figure out a way of getting the width! (See
other comment.)

 
   void Init(HWND hwnd);
@@ -33,6 +34,9 @@ class MinimizeButtonMetrics {
   void OnHWNDActivated();
 
  private:
+  static int GetCaptionButtonDimensionInDIPs(int caption_metric,
+                                             int frame_metric);
+
   // Gets the value for GetMinimizeButtonOffsetX(), caching if found.
   int GetAndCacheMinimizeButtonOffsetX() const;