Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(624)

Unified Diff: chrome/browser/ui/views/autofill/autofill_dialog_views.cc

Issue 17511006: payments dialog: fix legal document preferred size propagation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/autofill/autofill_dialog_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
index 23df6d72338220f6c5ee3d90220c4c54d957913a..c3a6315e0238f246cf817d5180d839b50e55963c 100644
--- a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
@@ -311,20 +311,22 @@ class DetailsContainerView : public views::View {
DISALLOW_COPY_AND_ASSIGN(DetailsContainerView);
};
-// ButtonStripView wraps the Autocheckout progress bar and the "[X] Save details
-// in Chrome" checkbox and listens for visibility changes.
-class ButtonStripView : public views::View {
+// A view that propagates visibility and preferred size changes.
+class LayoutPropagationView : public views::View {
public:
- ButtonStripView() {}
- virtual ~ButtonStripView() {}
+ LayoutPropagationView() {}
+ virtual ~LayoutPropagationView() {}
protected:
virtual void ChildVisibilityChanged(views::View* child) OVERRIDE {
PreferredSizeChanged();
}
+ virtual void ChildPreferredSizeChanged(views::View* child) OVERRIDE {
+ PreferredSizeChanged();
+ }
private:
- DISALLOW_COPY_AND_ASSIGN(ButtonStripView);
+ DISALLOW_COPY_AND_ASSIGN(LayoutPropagationView);
};
} // namespace
@@ -1217,7 +1219,7 @@ views::View* AutofillDialogViews::CreateTitlebarExtraView() {
}
views::View* AutofillDialogViews::CreateFootnoteView() {
- footnote_view_ = new views::View();
+ footnote_view_ = new LayoutPropagationView();
footnote_view_->SetLayoutManager(
new views::BoxLayout(views::BoxLayout::kVertical,
kLegalDocPadding,
@@ -1369,7 +1371,7 @@ void AutofillDialogViews::StyledLabelLinkClicked(const ui::Range& range,
}
void AutofillDialogViews::InitChildViews() {
- button_strip_extra_view_ = new ButtonStripView();
+ button_strip_extra_view_ = new LayoutPropagationView();
button_strip_extra_view_->SetLayoutManager(
new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0, 0));
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698