rAc: fix appearance of CVC input in autofill suggestion

chrome/browser/ui/views/autofill/autofill_dialog_views.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 "chrome/browser/ui/views/autofill/autofill_dialog_views.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 134 matching lines @@
     canvas->DrawPath(arrow, stroke_paint);
   }
 }
 
 // This class handles layout for the first row of a SuggestionView.
 // It exists to circumvent shortcomings of GridLayout and BoxLayout (namely that
 // the former doesn't fully respect child visibility, and that the latter won't
 // expand a single child).
 class SectionRowView : public views::View {
  public:
-  SectionRowView() {}
+  SectionRowView() {
+    set_border(views::Border::CreateEmptyBorder(10, 0, 0, 0));
+  }
+
   virtual ~SectionRowView() {}
 
   // views::View implementation:
   virtual gfx::Size GetPreferredSize() OVERRIDE {
     int height = 0;
     int width = 0;
     for (int i = 0; i < child_count(); ++i) {
       if (child_at(i)->visible()) {
         if (width > 0)
           width += kAroundTextPadding;
 
         gfx::Size size = child_at(i)->GetPreferredSize();
         height = std::max(height, size.height());
         width += size.width();
       }
     }
 
-    return gfx::Size(width, height);
+    gfx::Insets insets = GetInsets();
+    return gfx::Size(width + insets.width(), height + insets.height());
   }
 
   virtual void Layout() OVERRIDE {
     const gfx::Rect bounds = GetContentsBounds();
 
     // Icon is left aligned.
     int start_x = bounds.x();
     views::View* icon = child_at(0);
     if (icon->visible()) {
       icon->SizeToPreferredSize();
@@ skipping 875 matching lines @@
   SectionRowView* label_container = new SectionRowView();
   AddChildView(label_container);
 
   // Label and icon.
   label_container->AddChildView(icon_);
   label_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   label_container->AddChildView(label_);
 
   // TODO(estade): get the sizing and spacing right on this textfield.
   decorated_->SetVisible(false);
-  decorated_->set_default_width_in_chars(10);
+  decorated_->set_default_width_in_chars(15);
   label_container->AddChildView(decorated_);
 
   // TODO(estade): need to get the line height right.
   label_line_2_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   label_line_2_->SetVisible(false);
   label_line_2_->SetMultiLine(true);
   AddChildView(label_line_2_);
 
-  // TODO(estade): do something about this '2'.
-  SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 2, 0));
+  SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
 }
 
 AutofillDialogViews::SuggestionView::~SuggestionView() {}
 
 gfx::Size AutofillDialogViews::SuggestionView::GetPreferredSize() {
   // There's no preferred width. The parent's layout should get the preferred
   // height from GetHeightForWidth().
   return gfx::Size();
 }
 
@@ skipping 1330 matching lines @@
 AutofillDialogViews::DetailsGroup::DetailsGroup(DialogSection section)
     : section(section),
       container(NULL),
       manual_input(NULL),
       suggested_info(NULL),
       suggested_button(NULL) {}
 
 AutofillDialogViews::DetailsGroup::~DetailsGroup() {}
 
 }  // namespace autofill