Adjust menu item sizes for touch / non-touch according to spec.

ui/views/controls/menu/menu_config.cc

 // 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.
 
 #include "ui/views/controls/menu/menu_config.h"
 
 #include "build/build_config.h"
 #include "ui/base/layout.h"
 
 namespace views {
@@ skipping 23 matching lines @@
       gutter_width(0),
       separator_height(6),
       render_gutter(false),
       show_mnemonics(false),
       scroll_arrow_height(3),
       label_to_accelerator_padding(10),
       item_min_height(0),
       show_accelerators(true),
       always_use_icon_to_label_padding(false),
       align_arrow_and_shortcut(false) {
-  // Use 40px tall menu items when running in touch optimized mode.
-  if (ui::GetDisplayLayout() == ui::LAYOUT_TOUCH) {

[sky, 2012/08/10 22:17:25]

This change effects windows too. Do we care?

[Mr4D (OOO till 08-26), 2012/08/10 22:57:48]

For Windows: I am under the impression that we want to use the same layout now
everywhere. The only difference for Windows is that we have "real buttons" in
the wrench menu.

-    item_top_margin = item_no_icon_top_margin = 12;
-    item_bottom_margin = item_no_icon_bottom_margin = 13;
-  }
 }
 
 MenuConfig::~MenuConfig() {}
 
 void MenuConfig::Reset() {
   delete config_instance;
   config_instance = NULL;
 }
 
 // static
 const MenuConfig& MenuConfig::instance() {
   if (!config_instance)
     config_instance = Create();
   return *config_instance;
 }
 
 }  // namespace views