Index: chrome/browser/views/clear_browsing_data.cc |
=================================================================== |
--- chrome/browser/views/clear_browsing_data.cc (revision 3391) |
+++ chrome/browser/views/clear_browsing_data.cc (working copy) |
@@ -104,101 +104,98 @@ |
//////////////////////////////////////////////////////////////////////////////// |
// ClearBrowsingDataView, ChromeViews::View implementation: |
-void ClearBrowsingDataView::GetPreferredSize(CSize *out) { |
- DCHECK(out); |
- *out = ChromeViews::Window::GetLocalizedContentsSize( |
+gfx::Size ClearBrowsingDataView::GetPreferredSize() { |
+ return gfx::Size(ChromeViews::Window::GetLocalizedContentsSize( |
IDS_CLEARDATA_DIALOG_WIDTH_CHARS, |
- IDS_CLEARDATA_DIALOG_HEIGHT_LINES).ToSIZE(); |
+ IDS_CLEARDATA_DIALOG_HEIGHT_LINES)); |
} |
void ClearBrowsingDataView::Layout() { |
- CSize panel_size; |
- GetPreferredSize(&panel_size); |
+ gfx::Size panel_size = GetPreferredSize(); |
- CSize sz; |
- |
// Delete All label goes to the top left corner. |
- delete_all_label_->GetPreferredSize(&sz); |
+ gfx::Size sz = delete_all_label_->GetPreferredSize(); |
delete_all_label_->SetBounds(kPanelHorizMargin, kPanelVertMargin, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
// Check-boxes go beneath it (with a little indentation). |
- del_history_checkbox_->GetPreferredSize(&sz); |
+ sz = del_history_checkbox_->GetPreferredSize(); |
del_history_checkbox_->SetBounds(2 * kPanelHorizMargin, |
delete_all_label_->y() + |
delete_all_label_->height() + |
kRelatedControlVerticalSpacing, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
- del_downloads_checkbox_->GetPreferredSize(&sz); |
+ sz = del_downloads_checkbox_->GetPreferredSize(); |
del_downloads_checkbox_->SetBounds(2 * kPanelHorizMargin, |
del_history_checkbox_->y() + |
del_history_checkbox_->height() + |
kRelatedControlVerticalSpacing, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
- del_cache_checkbox_->GetPreferredSize(&sz); |
+ sz = del_cache_checkbox_->GetPreferredSize(); |
del_cache_checkbox_->SetBounds(2 * kPanelHorizMargin, |
del_downloads_checkbox_->y() + |
del_downloads_checkbox_->height() + |
kRelatedControlVerticalSpacing, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
- del_cookies_checkbox_->GetPreferredSize(&sz); |
+ sz = del_cookies_checkbox_->GetPreferredSize(); |
del_cookies_checkbox_->SetBounds(2 * kPanelHorizMargin, |
del_cache_checkbox_->y() + |
del_cache_checkbox_->height() + |
kRelatedControlVerticalSpacing, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
- del_passwords_checkbox_->GetPreferredSize(&sz); |
+ sz = del_passwords_checkbox_->GetPreferredSize(); |
del_passwords_checkbox_->SetBounds(2 * kPanelHorizMargin, |
del_cookies_checkbox_->y() + |
del_cookies_checkbox_->height() + |
kRelatedControlVerticalSpacing, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
// Time period label is next below the combo boxes. |
- time_period_label_->GetPreferredSize(&sz); |
+ sz = time_period_label_->GetPreferredSize(); |
time_period_label_->SetBounds(kPanelHorizMargin, |
del_passwords_checkbox_->y() + |
del_passwords_checkbox_->height() + |
kRelatedControlVerticalSpacing + |
kExtraMarginForTimePeriodLabel, |
- sz.cx, sz.cy); |
+ sz.width(), sz.height()); |
// Time period combo box goes on the right of the label, and we align it |
// vertically to the label as well. |
- int label_y_size = sz.cy; |
- time_period_combobox_->GetPreferredSize(&sz); |
+ int label_y_size = sz.height(); |
+ sz = time_period_combobox_->GetPreferredSize(); |
time_period_combobox_->SetBounds(time_period_label_->x() + |
time_period_label_->width() + |
kRelatedControlVerticalSpacing, |
time_period_label_->y() - |
- ((sz.cy - label_y_size) / 2), |
- sz.cx, sz.cy); |
+ ((sz.height() - label_y_size) / 2), |
+ sz.width(), sz.height()); |
// Get the y-coordinate of our parent so we can position the throbber and |
// status message at the bottom of the panel. |
CRect parent_bounds; |
GetParent()->GetLocalBounds(&parent_bounds, false); |
- throbber_->GetPreferredSize(&sz); |
+ sz = throbber_->GetPreferredSize(); |
int throbber_topleft_x = kPanelHorizMargin; |
- int throbber_topleft_y = parent_bounds.bottom - sz.cy - |
+ int throbber_topleft_y = parent_bounds.bottom - sz.height() - |
kButtonVEdgeMargin - 3; |
- throbber_->SetBounds(throbber_topleft_x, throbber_topleft_y, sz.cx, sz.cy); |
+ throbber_->SetBounds(throbber_topleft_x, throbber_topleft_y, sz.width(), |
+ sz.height()); |
// The status label should be at the bottom of the screen, to the right of |
// the throbber. |
- status_label_.GetPreferredSize(&sz); |
+ sz = status_label_.GetPreferredSize(); |
int status_label_x = throbber_->x() + throbber_->width() + |
kRelatedControlHorizontalSpacing; |
status_label_.SetHorizontalAlignment(ChromeViews::Label::ALIGN_LEFT); |
status_label_.SetBounds(status_label_x, |
throbber_topleft_y + 1, |
- sz.cx, |
- sz.cy); |
+ sz.width(), |
+ sz.height()); |
} |
void ClearBrowsingDataView::ViewHierarchyChanged(bool is_add, |