Convert GetPreferredSize from:...

chrome/browser/views/about_chrome_view.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/views/about_chrome_view.h"
 
 #include <commdlg.h>
 
 #include "base/file_version_info.h"
 #include "base/string_util.h"
@@ skipping 171 matching lines @@
   terms_of_service_url_ =
       new ChromeViews::Link(l10n_util::GetString(IDS_TERMS_OF_SERVICE));
   AddChildView(terms_of_service_url_);
   terms_of_service_url_->SetController(this);
 #endif
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // AboutChromeView, ChromeViews::View implementation:
 
-void AboutChromeView::GetPreferredSize(CSize *out) {
-  DCHECK(out);
-  *out = ChromeViews::Window::GetLocalizedContentsSize(
+gfx::Size AboutChromeView::GetPreferredSize() {
+  gfx::Size prefsize(ChromeViews::Window::GetLocalizedContentsSize(
       IDS_ABOUT_DIALOG_WIDTH_CHARS,
-      IDS_ABOUT_DIALOG_HEIGHT_LINES).ToSIZE();
+      IDS_ABOUT_DIALOG_HEIGHT_LINES));
   // We compute the height of the dialog based on the size of the image (it
   // would be nice to not hard code this), the text in the about dialog and the
   // margins around the text.
-  out->cy += 145 + (kPanelVertMargin * 2);
+  prefsize.Enlarge(0, 145 + (kPanelVertMargin * 2));
   // TODO(beng): Eventually the image should be positioned such that hard-
   //             coding the width isn't necessary.  This breaks with fonts
   //             that are large and cause wrapping.
-  out->cx = 422;
+  prefsize.set_width(422);
+  return prefsize;
 }
 
 void AboutChromeView::Layout() {
-  CSize panel_size;
-  GetPreferredSize(&panel_size);
-
-  CSize sz;
+  gfx::Size panel_size = GetPreferredSize();
 
   // Background image for the dialog.
-  about_dlg_background_->GetPreferredSize(&sz);
-  int background_image_height = sz.cy;  // used to position main text below.
-  about_dlg_background_->SetBounds(0, 0, sz.cx, sz.cy);
+  gfx::Size sz = about_dlg_background_->GetPreferredSize();
+  // used to position main text below.
+  int background_image_height = sz.height();
+  about_dlg_background_->SetBounds(0, 0, sz.width(), sz.height());
 
   // First label goes to the top left corner.
-  about_title_label_->GetPreferredSize(&sz);
+  sz = about_title_label_->GetPreferredSize();
   about_title_label_->SetBounds(kPanelHorizMargin, kPanelVertMargin,
-                                sz.cx, sz.cy);
+                                sz.width(), sz.height());
 
   // Then we have the version number right below it.
-  version_label_->GetPreferredSize(&sz);
+  sz = version_label_->GetPreferredSize();
   version_label_->SetBounds(kPanelHorizMargin,
                             about_title_label_->y() +
                                 about_title_label_->height() +
                                 kRelatedControlVerticalSpacing,
                             kVersionFieldWidth,
-                            sz.cy);
+                            sz.height());
 
   // For the width of the main text label we want to use up the whole panel
   // width and remaining height, minus a little margin on each side.
   int y_pos = background_image_height + kRelatedControlVerticalSpacing;
-  sz.cx = panel_size.cx - 2 * kPanelHorizMargin;
+  sz.set_width(panel_size.width() - 2 * kPanelHorizMargin);
 
   // Draw the text right below the background image.
   copyright_label_->SetBounds(kPanelHorizMargin,
                               y_pos,
-                              sz.cx,
-                              sz.cy);
+                              sz.width(),
+                              sz.height());
 
   // Then the main_text_label.
   main_text_label_->SetBounds(kPanelHorizMargin,
                               copyright_label_->y() +
                                   copyright_label_->height(),
-                              sz.cx,
+                              sz.width(),
                               main_text_label_height_);
 
   // Position the URLs within the main label. The rects here are calculated when
   // we draw the strings. First position the Chromium URL within the main label.
   chromium_url_->SetBounds(chromium_url_rect_.x(),
                            chromium_url_rect_.y(),
                            chromium_url_rect_.width(),
                            chromium_url_rect_.height());
   // Then position the Open Source URL within the main label.
   open_source_url_->SetBounds(open_source_url_rect_.x(),
                               open_source_url_rect_.y(),
                               open_source_url_rect_.width(),
                               open_source_url_rect_.height());
 #if defined(GOOGLE_CHROME_BUILD)
   // Then position the TOS URL within the main label.
   terms_of_service_url_->SetBounds(terms_of_service_url_rect_.x(),
                                    terms_of_service_url_rect_.y(),
                                    terms_of_service_url_rect_.width(),
                                    terms_of_service_url_rect_.height());
 #endif
 
   // Get the y-coordinate of our parent so we can position the text left of the
   // buttons at the bottom.
   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());
 
   // This image is hidden (see ViewHierarchyChanged) and displayed on demand.
-  success_indicator_.GetPreferredSize(&sz);
+  sz = success_indicator_.GetPreferredSize();
   success_indicator_.SetBounds(throbber_topleft_x, throbber_topleft_y,
-                               sz.cx, sz.cy);
+                               sz.width(), sz.height());
 
   // This image is hidden (see ViewHierarchyChanged) and displayed on demand.
-  update_available_indicator_.GetPreferredSize(&sz);
+  sz = update_available_indicator_.GetPreferredSize();
   update_available_indicator_.SetBounds(throbber_topleft_x, throbber_topleft_y,
-                                        sz.cx, sz.cy);
+                                        sz.width(), sz.height());
 
   // This image is hidden (see ViewHierarchyChanged) and displayed on demand.
-  timeout_indicator_.GetPreferredSize(&sz);
+  sz = timeout_indicator_.GetPreferredSize();
   timeout_indicator_.SetBounds(throbber_topleft_x, throbber_topleft_y,
-                                sz.cx, sz.cy);
+                               sz.width(), sz.height());
 
   // The update label should be at the bottom of the screen, to the right of
   // the throbber. We specify width to the end of the dialog because it contains
   // variable length messages.
-  update_label_.GetPreferredSize(&sz);
+  sz = update_label_.GetPreferredSize();
   int update_label_x = throbber_->x() + throbber_->width() +
                        kRelatedControlHorizontalSpacing;
   update_label_.SetHorizontalAlignment(ChromeViews::Label::ALIGN_LEFT);
   update_label_.SetBounds(update_label_x,
                           throbber_topleft_y + 1,
                           parent_bounds.Width() - update_label_x,
-                          sz.cy);
+                          sz.height());
 }
 
 
 void AboutChromeView::Paint(ChromeCanvas* canvas) {
   ChromeViews::View::Paint(canvas);
 
   ChromeFont font =
     ResourceBundle::GetSharedInstance().GetFont(ResourceBundle::BaseFont);
 
   const gfx::Rect label_bounds = main_text_label_->bounds();
 
   ChromeViews::Link* link1 =
       chromium_url_appears_first_ ? chromium_url_ : open_source_url_;
   ChromeViews::Link* link2 =
       chromium_url_appears_first_ ? open_source_url_ : chromium_url_;
   gfx::Rect* rect1 = chromium_url_appears_first_ ?
       &chromium_url_rect_ : &open_source_url_rect_;
   gfx::Rect* rect2 = chromium_url_appears_first_ ?
       &open_source_url_rect_ : &chromium_url_rect_;
 
   // This struct keeps track of where to write the next word (which x,y
   // pixel coordinate). This struct is updated after drawing text and checking
   // if we need to wrap.
-  CSize position;
+  gfx::Size position;
   // Draw the first text chunk and position the Chromium url.
   DrawTextAndPositionUrl(canvas, main_label_chunk1_, link1,
                          rect1, &position, label_bounds, font);
   // Draw the second text chunk and position the Open Source url.
   DrawTextAndPositionUrl(canvas, main_label_chunk2_, link2,
                          rect2, &position, label_bounds, font);
   // Draw the third text chunk.
   DrawTextStartingFrom(canvas, main_label_chunk3_, &position, label_bounds,
                        font);
 
 #if defined(GOOGLE_CHROME_BUILD)
   // Insert a line break and some whitespace.
-  position.cx = 0;
-  position.cy += font.height() + kRelatedControlVerticalSpacing;
+  position.set_width(0);
+  position.Enlarge(0, font.height() + kRelatedControlVerticalSpacing);
 
   // And now the Terms of Service and position the TOS url.
   DrawTextAndPositionUrl(canvas, main_label_chunk4_, terms_of_service_url_,
                          &terms_of_service_url_rect_, &position, label_bounds,
                          font);
   // The last text chunk doesn't have a URL associated with it.
   DrawTextStartingFrom(canvas, main_label_chunk5_, &position, label_bounds,
                        font);
 #endif
 
   // Save the height so we can set the bounds correctly.
-  main_text_label_height_ = position.cy + font.height();
+  main_text_label_height_ = position.height() + font.height();
 }
 
 void AboutChromeView::DrawTextAndPositionUrl(ChromeCanvas* canvas,
                                              const std::wstring& text,
                                              ChromeViews::Link* link,
                                              gfx::Rect* rect,
-                                             CSize* position,
+                                             gfx::Size* position,
                                              const gfx::Rect& bounds,
                                              const ChromeFont& font) {
   DCHECK(canvas && link && rect && position);
   // Draw the text chunk.
   DrawTextStartingFrom(canvas, text, position, bounds, font);
 
   // And then position the link after it.
-  CSize sz;
-  link->GetPreferredSize(&sz);
-  WrapIfWordDoesntFit(sz.cx, font.height(), position, bounds);
-  *rect = gfx::Rect(position->cx, position->cy, sz.cx, sz.cy);
+  gfx::Size sz = link->GetPreferredSize();
+  WrapIfWordDoesntFit(sz.width(), font.height(), position, bounds);
+  *rect = gfx::Rect(position->width(), position->height(), sz.width(),
+                    sz.height());
 
   // Going from relative to absolute pixel coordinates again.
   rect->Offset(bounds.x(), bounds.y());
   // And leave some space to draw the link in.
-  position->cx += sz.cx;
+  position->Enlarge(sz.width(), 0);
 }
 
 void AboutChromeView::DrawTextStartingFrom(ChromeCanvas* canvas,
                                            const std::wstring& text,
-                                           CSize* position,
+                                           gfx::Size* position,
                                            const gfx::Rect& bounds,
                                            const ChromeFont& font) {
   // Iterate through line breaking opportunities (which in English would be
   // spaces and such. This tells us where to wrap.
   WordIterator iter(text, WordIterator::BREAK_LINE);
   if (!iter.Init())
     return;
 
   int flags = (UILayoutIsRightToLeft() ?
                    ChromeCanvas::TEXT_ALIGN_RIGHT :
                    ChromeCanvas::TEXT_ALIGN_LEFT) |
               ChromeCanvas::MULTI_LINE |
               ChromeCanvas::HIDE_PREFIX;
 
   // Iterate over each word in the text, or put in a more locale-neutral way:
   // iterate to the next line breaking opportunity.
   while (iter.Advance()) {
     // Get the word and figure out the dimensions.
     std::wstring word = iter.GetWord();
     int w = font.GetStringWidth(word), h = font.height();
     canvas->SizeStringInt(word, font, &w, &h, flags);
 
     // If we exceed the boundaries, we need to wrap.
     WrapIfWordDoesntFit(w, font.height(), position, bounds);
 
     // Draw the word on the screen (mirrored if RTL locale).
     canvas->DrawStringInt(word, font, SK_ColorBLACK,
         main_text_label_->MirroredXCoordinateInsideView(
-            position->cx + bounds.x()),
-            position->cy + bounds.y(),
+            position->width() + bounds.x()),
+            position->height() + bounds.y(),
             w, h, flags);
 
     if (word.size() > 0 && word[word.size() - 1] == L'\x0a') {
       // When we come across '\n', we move to the beginning of the next line.
-      position->cx = 0;
-      position->cy += font.height();
+      position->set_width(0);
+      position->Enlarge(0, font.height());
     } else {
       // Otherwise, we advance position to the next word.
-      position->cx += w;
+      position->Enlarge(w, 0);
     }
   }
 }
 
 void AboutChromeView::WrapIfWordDoesntFit(int word_width,
                                           int font_height,
-                                          CSize* position,
+                                          gfx::Size* position,
                                           const gfx::Rect& bounds) {
-  if (position->cx + word_width > bounds.right()) {
-    position->cx = 0;
-    position->cy += font_height;
+  if (position->width() + word_width > bounds.right()) {
+    position->set_width(0);
+    position->Enlarge(0, font_height);
   }
 }
 
 void AboutChromeView::ViewHierarchyChanged(bool is_add,
                                            ChromeViews::View* parent,
                                            ChromeViews::View* child) {
   // Since we want the some of the controls to show up in the same visual row
   // as the buttons, which are provided by the framework, we must add the
   // buttons to the non-client view, which is the parent of this view.
   // Similarly, when we're removed from the view hierarchy, we must take care
@@ skipping 238 matching lines @@
 
   // We have updated controls on the parent, so we need to update its layout.
   View* parent = GetParent();
   parent->Layout();
 
   // Check button may have appeared/disappeared. We cannot call this during
   // ViewHierarchyChanged because the |window()| pointer hasn't been set yet.
   if (window())
     GetDialogClientView()->UpdateDialogButtons();
 }