Last round of cleaning up MD layout constants.

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

Index: chrome/browser/ui/views/frame/browser_non_client_frame_view.h
diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view.h b/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
index 9fbced7c6deb1f5173553bf01d2f972a30acd6e2..a9ffe6094a2d130ec5a77a69d466e4be4cb6d6bc 100644
--- a/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
+++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
@@ -64,6 +64,9 @@ class BrowserNonClientFrameView : public views::NonClientFrameView,
   // Overriden from views::View.
   void VisibilityChanged(views::View* starting_from, bool is_visible) override;
 
+  // The padding on the left, right, and bottom of the avatar icon.
+  static constexpr int kAvatarIconPadding = 4;

[Peter Kasting, 2016/12/08 20:57:49]

Nit: Goes above constructor (see
http://google.github.io/styleguide/cppguide.html#Declaration_Order )

[Evan Stade, 2016/12/13 01:31:16]

thanks, for some reason my understanding stopped at "Within each section,
generally prefer grouping similar kinds of declarations together"

+
  protected:
   // Whether the frame should be painted with theming.
   // By default, tabbed browser windows are themed but popup and app windows are