Reflow of the profile items in desktop user menu

ui/views/layout/layout_constants.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 UI_VIEWS_LAYOUT_LAYOUT_CONSTANTS_H_
 #define UI_VIEWS_LAYOUT_LAYOUT_CONSTANTS_H_
 
 #include "build/build_config.h"
 #include "ui/views/layout/grid_layout.h"
 
 // This file contains some constants we use to implement our standard panel
 // layout.
 // see: spec 21/4
 
 namespace views {
 
 // Left or right margin.
 const int kPanelHorizMargin = 13;
 
 // Top or bottom margin.
 const int kPanelVertMargin = 13;
 
 // When several controls are aligned vertically, the baseline should be spaced
 // by the following number of pixels.
 const int kPanelVerticalSpacing = 32;
 
 // Vertical spacing between sub UI.
 const int kPanelSubVerticalSpacing = 24;
 
+// Vertical spacing between the edge of the menu and the top or bottom of the
+// menu items in material design.

[Evan Stade, 2016/06/10 17:39:46]

I have no idea what menu this comment is referring to

[Jane, 2016/06/10 18:49:16]

I thought this would be a common constant used by material design menus in
general; but I definitely can be more specific for now. Done.

+const int kMaterialVerticalMargin = 16;

[Evan Stade, 2016/06/10 17:39:46]

this is a rather unhelpful name. Pretend we're in the world where material is
default. In that world this becomes just kVerticalMargin.

[Jane, 2016/06/10 18:49:16]

Same thought but this name does need more details. Thanks for the suggestion and
let me know if this name works.

+
+// Horizontal spacing between the edge of the menu and the left or right of the
+// menu items in material design.
+const int kMaterialHorizontalMargin = 16;
+
 // Vertical spacing between a label and some control.
 const int kLabelToControlVerticalSpacing = 8;
 
 // Small horizontal spacing between controls that are logically related.
 const int kRelatedControlSmallHorizontalSpacing = 8;
 
 // Horizontal spacing between controls that are logically related.
 const int kRelatedControlHorizontalSpacing = 8;
 
 // Vertical spacing between controls that are logically related.
@@ skipping 43 matching lines @@
 // Horizontal alignment used for a control's introductory label in a GridLayout.
 #if defined(OS_MACOSX)
 const GridLayout::Alignment kControlLabelGridAlignment = GridLayout::TRAILING;
 #else
 const GridLayout::Alignment kControlLabelGridAlignment = GridLayout::LEADING;
 #endif
 
 }  // namespace views
 
 #endif  // UI_VIEWS_LAYOUT_LAYOUT_CONSTANTS_H_