Make some menu margins consistent across platforms

Bug: 850080
Change-Id: Iec8fdc70bb943fa153a4ffbacadda9ea3478370e
Reviewed-on: https://chromium-review.googlesource.com/1088755Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Commit-Queue: Fabio Tirelo <ftirelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564988}

ui/views/controls/menu/menu_config.cc

@@ -23,11 +23,11 @@ MenuConfig::MenuConfig()
       minimum_text_item_height(0),
       minimum_container_item_height(0),
       minimum_menu_width(0),
-      item_left_margin(10),
+      item_left_margin(8),
       touchable_item_left_margin(16),
-      label_to_arrow_padding(10),
+      label_to_arrow_padding(8),
       arrow_to_edge_padding(5),
-      icon_to_label_padding(10),
+      icon_to_label_padding(8),
       touchable_icon_to_label_padding(16),
       touchable_icon_size(20),
       touchable_icon_color(SkColorSetRGB(0x5F, 0x63, 0x60)),
@@ -44,7 +44,7 @@ MenuConfig::MenuConfig()
       show_mnemonics(false),
       use_mnemonics(true),
       scroll_arrow_height(3),
-      label_to_minor_text_padding(10),
+      label_to_minor_text_padding(8),
       item_min_height(0),
       actionable_submenu_arrow_to_edge_padding(14),
       actionable_submenu_width(37),

ui/views/controls/menu/menu_config_chromeos.cc

@@ -17,8 +17,6 @@ void MenuConfig::Init() {
   separator_spacing_height = 7;
   separator_lower_height = 8;
   separator_upper_height = 8;
-  label_to_arrow_padding = 20;
-  label_to_minor_text_padding = 20;
   always_use_icon_to_label_padding = true;
   align_arrow_and_shortcut = true;
   offset_context_menus = true;

ui/views/controls/menu/menu_config_mac.mm

@@ -20,10 +20,8 @@ void InitMaterialMenuConfig(views::MenuConfig* config) {
   config->minimum_text_item_height = 32;
   config->minimum_container_item_height = 48;
   config->minimum_menu_width = 320;
-  config->item_left_margin = 8;
   config->label_to_arrow_padding = 0;
   config->arrow_to_edge_padding = 16;
-  config->icon_to_label_padding = 8;
   config->check_width = 16;
   config->check_height = 16;
   config->arrow_width = 8;
@@ -32,7 +30,6 @@ void InitMaterialMenuConfig(views::MenuConfig* config) {
   config->separator_upper_height = 9;
   config->separator_spacing_height = 9;
   config->separator_thickness = 1;
-  config->label_to_minor_text_padding = 8;
   config->align_arrow_and_shortcut = true;
   config->use_outer_border = false;
   config->icons_in_label = true;