Add ability to compute text intercepts to Font

third_party/WebKit/Source/platform/fonts/Font.h

Index: third_party/WebKit/Source/platform/fonts/Font.h
diff --git a/third_party/WebKit/Source/platform/fonts/Font.h b/third_party/WebKit/Source/platform/fonts/Font.h
index bff7ab2b0ded9a9ba7b53770ea2e1d11a38d1433..3ab36edcc2db0c466e1c080752d3d7b958c0954e 100644
--- a/third_party/WebKit/Source/platform/fonts/Font.h
+++ b/third_party/WebKit/Source/platform/fonts/Font.h
@@ -101,6 +101,20 @@ class PLATFORM_EXPORT Font {
                          float deviceScaleFactor,
                          const SkPaint&) const;
 
+  // This method follows the behavior and allocation model of the underlying
+  // Skia function: Return the number of intervals that intersect the intercept
+  // along the axis of the advance.  The return count is zero or a multiple of
+  // two, and is at most the number of glyphs * 2 in TextRun part of

[pdr., 2016/10/14 04:40:56]

Super nit: Add "the", remove the second period.
"...number of glyphs * 2 in [the] TextRun part of TextRunPaintInfo."

[drott, 2016/10/14 15:16:44]

I changed the signature of this method and the comment. So this is implicitly
addressed.

+  // TextRunPaintInfo.. The caller may pass nullptr for intervals to determine
+  // the size of the interval array.
+  // TODO(drott): crbug.com/655154 Fix this for upright in vertical.
+  int getTextIntercepts(SkCanvas*,

[f(malita), 2016/10/13 18:09:19]

SkCanvas param doesn't seem needed.

[drott, 2016/10/14 15:15:34]

Ah true, thanks - I initially thought I needed it, that's also why I placed this
on GraphicsContext. Good then.

+                        const TextRunPaintInfo&,
+                        float deviceScaleFactor,
+                        const SkPaint&,
+                        const SkScalar* bounds,
+                        SkScalar* intervals) const;

[f(malita), 2016/10/13 18:09:19]

I'm not necessarily opposed to following Skia's signature here, but it would
also make sense to consider a higher-level, Blink-oriented API.  How about
something like

struct Intercept { float mBegin, mEnd; };

bool getTextIntercepts(const TextRunPaintInfo&,
                       float deviceScaleFactor,
                       const SkPaint&,
                       Vector<Intercept>& intercepts,
                       std::tuple<float, float>& bounds);

Then the caller wouldn't have to invoke twice, but instead the impl can hide
that detail and resize the out vector as needed.

[drott, 2016/10/14 15:15:34]

Good idea, looks better and less alien on the Blink side after changing that.

+
   // Glyph bounds will be the minimum rect containing all glyph strokes, in
   // coordinates using (<text run x position>, <baseline position>) as the
   // origin.