Add Vector2d classes that represent offsets, instead of using Point.

ui/gfx/render_text_mac.cc

 // Copyright (c) 2012 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 "ui/gfx/render_text_mac.h"
 
 #include <ApplicationServices/ApplicationServices.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 214 matching lines @@
     }
   }
 }
 
 void RenderTextMac::ComputeRuns() {
   DCHECK(line_);
 
   CFArrayRef ct_runs = CTLineGetGlyphRuns(line_);
   const CFIndex ct_runs_count = CFArrayGetCount(ct_runs);
 
-  Point offset(GetTextOrigin());
   // Skia will draw glyphs with respect to the baseline.
-  offset.Offset(0, common_baseline_);
-
-  const SkScalar x = SkIntToScalar(offset.x());
-  const SkScalar y = SkIntToScalar(offset.y());
-  SkPoint run_origin = SkPoint::Make(offset.x(), offset.y());
+  const SkScalar x = SkIntToScalar(GetTextOffset().x());

[Peter Kasting, 2012/10/31 01:04:41]

Nit: FWIW, I had intended to still have a separate Vector2d temp, I only meant
for the Grow() call to be inlined.  GetTextOffset() may not be cheap.

[danakj, 2012/10/31 16:55:46]

Done.

+  const SkScalar y = SkIntToScalar(GetTextOffset().y() + common_baseline_);
+  SkPoint run_origin = SkPoint::Make(x, y);
 
   const CFRange empty_cf_range = CFRangeMake(0, 0);
   for (CFIndex i = 0; i < ct_runs_count; ++i) {
     CTRunRef ct_run =
         base::mac::CFCast<CTRunRef>(CFArrayGetValueAtIndex(ct_runs, i));
     const size_t glyph_count = CTRunGetGlyphCount(ct_run);
     const double run_width =
         CTRunGetTypographicBounds(ct_run, empty_cf_range, NULL, NULL, NULL);
     if (glyph_count == 0) {
       run_origin.offset(run_width, 0);
@@ skipping 63 matching lines @@
     run_origin.offset(run_width, 0);
   }
   runs_valid_ = true;
 }
 
 RenderText* RenderText::CreateInstance() {
   return new RenderTextMac;
 }
 
 }  // namespace gfx