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

Unified Diff: chrome/views/message_box_view.cc

Issue 7344: Convert GetPreferredSize from:... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 12 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: chrome/views/message_box_view.cc
===================================================================
--- chrome/views/message_box_view.cc (revision 3391)
+++ chrome/views/message_box_view.cc (working copy)
@@ -119,9 +119,9 @@
GridLayout* layout = CreatePanelGridLayout(this);
SetLayoutManager(layout);
- CSize icon_size;
+ gfx::Size icon_size;
if (icon_)
- icon_->GetPreferredSize(&icon_size);
+ icon_size = icon_->GetPreferredSize();
// Add the column set for the message displayed at the top of the dialog box.
// And an icon, if one has been set.
@@ -129,7 +129,8 @@
ColumnSet* column_set = layout->AddColumnSet(message_column_view_set_id);
if (icon_) {
column_set->AddColumn(GridLayout::LEADING, GridLayout::LEADING, 0,
- GridLayout::FIXED, icon_size.cx, icon_size.cy);
+ GridLayout::FIXED, icon_size.width(),
+ icon_size.height());
column_set->AddPaddingColumn(0, kUnrelatedControlHorizontalSpacing);
}
column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1,
@@ -141,7 +142,7 @@
column_set = layout->AddColumnSet(textfield_column_view_set_id);
if (icon_) {
column_set->AddPaddingColumn(0,
- icon_size.cx + kUnrelatedControlHorizontalSpacing);
+ icon_size.width() + kUnrelatedControlHorizontalSpacing);
}
column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1,
GridLayout::USE_PREF, 0, 0);
@@ -153,7 +154,7 @@
column_set = layout->AddColumnSet(checkbox_column_view_set_id);
if (icon_) {
column_set->AddPaddingColumn(0,
- icon_size.cx + kUnrelatedControlHorizontalSpacing);
+ icon_size.width() + kUnrelatedControlHorizontalSpacing);
}
column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1,
GridLayout::USE_PREF, 0, 0);

Powered by Google App Engine
This is Rietveld 408576698