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

Unified Diff: ui/views/controls/message_box_view.cc

Issue 10933085: Update ConstrainedWindowViews appearance according to mock (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Browser test fixes Created 8 years, 2 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
Index: ui/views/controls/message_box_view.cc
diff --git a/ui/views/controls/message_box_view.cc b/ui/views/controls/message_box_view.cc
index 5d5e909f4fe7c42d874b22fd80017be691e0f766..fcde521a1910a97fdd8b1780c48a5d28155195ff 100644
--- a/ui/views/controls/message_box_view.cc
+++ b/ui/views/controls/message_box_view.cc
@@ -66,7 +66,13 @@ namespace views {
MessageBoxView::InitParams::InitParams(const string16& message)
: options(NO_OPTIONS),
message(message),
- message_width(kDefaultMessageWidth) {
+ message_width(kDefaultMessageWidth),
+ top_inset(kPanelVertMargin),
+ bottom_inset(kPanelVertMargin),
+ left_inset(kPanelHorizMargin),
+ right_inset(kPanelHorizMargin),
+ inter_row_vertical_spacing(kRelatedControlVerticalSpacing)
+{
}
MessageBoxView::InitParams::~InitParams() {
@@ -188,12 +194,19 @@ void MessageBoxView::Init(const InitParams& params) {
prompt_field_->SetText(params.default_prompt);
}
+ top_inset_ = params.top_inset;
+ bottom_inset_ = params.bottom_inset;
+ left_inset_ = params.left_inset;
+ right_inset_ = params.right_inset;
+ inter_row_vertical_spacing_ = params.inter_row_vertical_spacing;
+
ResetLayoutManager();
}
void MessageBoxView::ResetLayoutManager() {
// Initialize the Grid Layout Manager used for this dialog box.
GridLayout* layout = GridLayout::CreatePanel(this);
+ layout->SetInsets(top_inset_, bottom_inset_, left_inset_, right_inset_);
SetLayoutManager(layout);
gfx::Size icon_size;
@@ -249,18 +262,18 @@ void MessageBoxView::ResetLayoutManager() {
}
if (prompt_field_) {
- layout->AddPaddingRow(0, kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, inter_row_vertical_spacing_);
layout->StartRow(0, textfield_column_view_set_id);
layout->AddView(prompt_field_);
}
if (checkbox_) {
- layout->AddPaddingRow(0, kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, inter_row_vertical_spacing_);
layout->StartRow(0, checkbox_column_view_set_id);
layout->AddView(checkbox_);
}
- layout->AddPaddingRow(0, kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, inter_row_vertical_spacing_);
}
} // namespace views

Powered by Google App Engine
This is Rietveld 408576698