Modernize autofill popup design

chrome/browser/ui/autofill/autofill_popup_layout_model.cc

 // Copyright (c) 2016 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 "chrome/browser/ui/autofill/autofill_popup_layout_model.h"
 
 #include <algorithm>
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/autofill/autofill_popup_view.h"
 #include "chrome/browser/ui/autofill/popup_constants.h"
 #include "components/autofill/core/browser/popup_item_ids.h"
 #include "components/autofill/core/browser/suggestion.h"
 #include "components/autofill/core/common/autofill_util.h"
 #include "grit/components_scaled_resources.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/geometry/rect_conversions.h"
 
 namespace autofill {
 
 namespace {
 
 // The vertical height of each row in pixels.
-const size_t kRowHeight = 24;
+const size_t kRowHeight = 44;

[Evan Stade, 2016/01/25 23:19:38]

I would recommend reusing MenuConfig layout constants such as item_bottom_margin
instead of hardcoding some value which will match at most one platform

 
 // The vertical height of a separator in pixels.
 const size_t kSeparatorHeight = 1;
 
 const struct {
   const char* name;
   int id;
 } kDataResources[] = {
     {"americanExpressCC", IDR_AUTOFILL_CC_AMEX},
     {"dinersCC", IDR_AUTOFILL_CC_GENERIC},
@@ skipping 135 matching lines @@
 #endif
 
   return result;
 }
 
 const gfx::Rect AutofillPopupLayoutModel::RoundedElementBounds() const {
   return gfx::ToEnclosingRect(delegate_->element_bounds());
 }
 
 }  // namespace autofill