| Index: chrome/browser/views/first_run_customize_view.cc
|
| ===================================================================
|
| --- chrome/browser/views/first_run_customize_view.cc (revision 3391)
|
| +++ chrome/browser/views/first_run_customize_view.cc (working copy)
|
| @@ -83,11 +83,10 @@
|
| quick_shortcut_cbox_->SetIsSelected(true);
|
| }
|
|
|
| -void FirstRunCustomizeView::GetPreferredSize(CSize *out) {
|
| - DCHECK(out);
|
| - *out = ChromeViews::Window::GetLocalizedContentsSize(
|
| +gfx::Size FirstRunCustomizeView::GetPreferredSize() {
|
| + return gfx::Size(ChromeViews::Window::GetLocalizedContentsSize(
|
| IDS_FIRSTRUNCUSTOMIZE_DIALOG_WIDTH_CHARS,
|
| - IDS_FIRSTRUNCUSTOMIZE_DIALOG_HEIGHT_LINES).ToSIZE();
|
| + IDS_FIRSTRUNCUSTOMIZE_DIALOG_HEIGHT_LINES));
|
| }
|
|
|
| void FirstRunCustomizeView::Layout() {
|
| @@ -96,63 +95,62 @@
|
| const int kVertSpacing = 8;
|
| const int kComboExtraPad = 8;
|
|
|
| - CSize canvas;
|
| - GetPreferredSize(&canvas);
|
| + gfx::Size canvas = GetPreferredSize();
|
|
|
| // Welcome label goes in to to the left. It does not go across the
|
| // entire window because the background gets busy on the right.
|
| - CSize pref_size;
|
| - main_label_->GetPreferredSize(&pref_size);
|
| + gfx::Size pref_size = main_label_->GetPreferredSize();
|
| main_label_->SetBounds(kPanelHorizMargin, kPanelVertMargin,
|
| - canvas.cx - pref_size.cx, pref_size.cy);
|
| + canvas.width() - pref_size.width(),
|
| + pref_size.height());
|
| AdjustDialogWidth(main_label_);
|
|
|
| int next_v_space = background_image()->y() +
|
| background_image()->height() + kPanelVertMargin;
|
|
|
| - import_cbox_->GetPreferredSize(&pref_size);
|
| + pref_size = import_cbox_->GetPreferredSize();
|
| import_cbox_->SetBounds(kPanelHorizMargin, next_v_space,
|
| - pref_size.cx, pref_size.cy);
|
| + pref_size.width(), pref_size.height());
|
|
|
| import_cbox_->SetIsSelected(true);
|
|
|
| int x_offset = import_cbox_->x() +
|
| import_cbox_->width();
|
|
|
| - import_from_combo_->GetPreferredSize(&pref_size);
|
| + pref_size = import_from_combo_->GetPreferredSize();
|
| import_from_combo_->SetBounds(x_offset,
|
| next_v_space +
|
| (import_cbox_->height() -
|
| - pref_size.cy) / 2,
|
| - pref_size.cx + kComboExtraPad,
|
| - pref_size.cy);
|
| + pref_size.height()) / 2,
|
| + pref_size.width() + kComboExtraPad,
|
| + pref_size.height());
|
|
|
| AdjustDialogWidth(import_from_combo_);
|
|
|
| next_v_space = import_cbox_->y() + import_cbox_->height() +
|
| kUnrelatedControlVerticalSpacing;
|
|
|
| - shortcuts_label_->GetPreferredSize(&pref_size);
|
| + pref_size = shortcuts_label_->GetPreferredSize();
|
| shortcuts_label_->SetBounds(kPanelHorizMargin, next_v_space,
|
| - pref_size.cx, pref_size.cy);
|
| + pref_size.width(), pref_size.height());
|
|
|
| AdjustDialogWidth(shortcuts_label_);
|
|
|
| next_v_space += shortcuts_label_->height() +
|
| kRelatedControlVerticalSpacing;
|
|
|
| - desktop_shortcut_cbox_->GetPreferredSize(&pref_size);
|
| + pref_size = desktop_shortcut_cbox_->GetPreferredSize();
|
| desktop_shortcut_cbox_->SetBounds(kPanelHorizMargin, next_v_space,
|
| - pref_size.cx, pref_size.cy);
|
| + pref_size.width(), pref_size.height());
|
|
|
| AdjustDialogWidth(desktop_shortcut_cbox_);
|
|
|
| next_v_space += desktop_shortcut_cbox_->height() +
|
| kRelatedControlVerticalSpacing;
|
|
|
| - quick_shortcut_cbox_->GetPreferredSize(&pref_size);
|
| + pref_size = quick_shortcut_cbox_->GetPreferredSize();
|
| quick_shortcut_cbox_->SetBounds(kPanelHorizMargin, next_v_space,
|
| - pref_size.cx, pref_size.cy);
|
| + pref_size.width(), pref_size.height());
|
|
|
| AdjustDialogWidth(quick_shortcut_cbox_);
|
| }
|
|
|