Fixing autocheckout progress bar repaint bug.

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/utf_string_conversions.h"
@@ skipping 619 matching lines @@
     const string16& placeholder_text,
     const gfx::ImageSkia& icon) {
   decorated_->textfield()->set_placeholder_text(placeholder_text);
   decorated_->textfield()->SetIcon(icon);
   decorated_->SetVisible(true);
   // The textfield will increase the height of the first row and cause the
   // label to be aligned properly, so the border is not necessary.
   label_->set_border(NULL);
 }
 
+// AutofilDialogViews::ButtonStripView ---------------------------------
+
+AutofillDialogViews::ButtonStripView::ButtonStripView() {}
+AutofillDialogViews::ButtonStripView::~ButtonStripView() {}
+
+void AutofillDialogViews::ButtonStripView::ChildVisibilityChanged(
+    views::View* child) {
+  PreferredSizeChanged();
+}
+
 // AutofilDialogViews::AutocheckoutProgressBar ---------------------------------
 
 AutofillDialogViews::AutocheckoutProgressBar::AutocheckoutProgressBar() {}
+AutofillDialogViews::AutocheckoutProgressBar::~AutocheckoutProgressBar() {}
 
 gfx::Size AutofillDialogViews::AutocheckoutProgressBar::GetPreferredSize() {
   return gfx::Size(kAutocheckoutProgressBarWidth,
                    kAutocheckoutProgressBarHeight);
 }
 
 // AutofillDialogView ----------------------------------------------------------
 
 // static
 AutofillDialogView* AutofillDialogView::Create(
@@ skipping 74 matching lines @@
 
 void AutofillDialogViews::UpdateAccountChooser() {
   account_chooser_->Update();
 
   // Update legal documents for the account.
   if (footnote_view_) {
     string16 text = controller_->LegalDocumentsText();
     if (text.empty()) {
       footnote_view_->SetVisible(false);
     } else {
       footnote_view_->SetVisible(true);

[Evan Stade, 2013/05/07 22:25:11]

and I think you'll need to move this to the end of the block here

[Dan Beam, 2013/05/08 00:54:10]

Done.

       legal_document_view_->SetText(text);
 
       const std::vector<ui::Range>& link_ranges =
           controller_->LegalDocumentLinks();
       for (size_t i = 0; i < link_ranges.size(); ++i) {
         legal_document_view_->AddStyleRange(
             link_ranges[i],
             views::StyledLabel::RangeStyleInfo::CreateForLink());
       }
     }
@@ skipping 341 matching lines @@
   DCHECK(group);
   ValidateGroup(group, AutofillDialogController::VALIDATE_EDIT);
 }
 
 void AutofillDialogViews::StyledLabelLinkClicked(const ui::Range& range,
                                                  int event_flags) {
   controller_->LegalDocumentLinkClicked(range);
 }
 
 void AutofillDialogViews::InitChildViews() {
-  button_strip_extra_view_ = new views::View();
+  button_strip_extra_view_ = new ButtonStripView();
   button_strip_extra_view_->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0, 0));
 
   save_in_chrome_checkbox_ =
       new views::Checkbox(controller_->SaveLocallyText());
   button_strip_extra_view_->AddChildView(save_in_chrome_checkbox_);
 
   autocheckout_progress_bar_view_ = new views::View();
   autocheckout_progress_bar_view_->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
@@ skipping 471 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