[LayoutNG] Add NG-specific DrawText support to GraphicsContext

[LayoutNG] Add NG-specific DrawText support to GraphicsContext

Add LayoutNG versions of the DrawText and DrawEmphasisMarks methods that
operate on TextFragmentPaintInfo instances (wrapping text fragment data)
instead of TextRunPaintInfo instances. Reuses impl when at all possible.

Also adds support for bloberizing emphasis marks for text fragments with
the addition of a FillTextEmphasisGlyphs StringView/ShapeResult version. 

BUG=714962
R=kojii@chromium.org

Review-Url: https://codereview.chromium.org/2845883002
Cr-Commit-Position: refs/heads/master@{#467592}
Committed: https://chromium.googlesource.com/chromium/src/+/67b9f6e761fba98d3a3e80edec863d2b97ff79a5

[eae, 2017-04-27 03:26:44]

Description was changed from

==========
[LayoutNG] Add NG-specific DrawText support to GraphicsContext

Add LayoutNG versions of the DrawText and DrawEmphasisMarks methods that
operate on TextFragmentPaintInfo instances (wrapping text fragment data)
instead of TextRunPaintInfo instances. Reuses logic when at all
possible.

BUG=714962
R=kojii@chromium.org
==========

to

==========
[LayoutNG] Add NG-specific DrawText support to GraphicsContext

Add LayoutNG versions of the DrawText and DrawEmphasisMarks methods that
operate on TextFragmentPaintInfo instances (wrapping text fragment data)
instead of TextRunPaintInfo instances. Reuses impl when at all possible.

Also adds support for bloberizing emphasis marks for text fragments with
the addition of a FillTextEmphasisGlyphs StringView/ShapeResult version. 

BUG=714962
R=kojii@chromium.org
==========

[eae, 2017-04-27 03:27:03]

The CQ bit was checked by eae@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-27 03:27:44]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[eae, 2017-04-27 03:30:32]

https://codereview.chromium.org/2845883002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/graphics/GraphicsContext.h (right):

https://codereview.chromium.org/2845883002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/graphics/GraphicsContext.h:240: void
DrawText(const Font&, const TextFragmentPaintInfo&, const FloatPoint&);
I tried templetazing the public methods themselves but couldn't get it to link
reliably across all platforms, even with explicit template instantiation.

This way it works reliably and it makes it very clear what types are supported.

[kojii, 2017-04-27 03:54:12]

lgtm, great!

[commit-bot: I haz the power, 2017-04-27 05:15:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-27 05:15:53]

Dry run: This issue passed the CQ dry run.

[kojii, 2017-04-27 05:27:30]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-04-27 05:27:57]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-27 05:32:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493270850531340, "parent_rev":
"b7bd7c79d7c9e89dc91746bba19a40ef8ec3b116", "commit_rev":
"67b9f6e761fba98d3a3e80edec863d2b97ff79a5"}

[commit-bot: I haz the power, 2017-04-27 05:33:56]

Description was changed from

==========
[LayoutNG] Add NG-specific DrawText support to GraphicsContext

Add LayoutNG versions of the DrawText and DrawEmphasisMarks methods that
operate on TextFragmentPaintInfo instances (wrapping text fragment data)
instead of TextRunPaintInfo instances. Reuses impl when at all possible.

Also adds support for bloberizing emphasis marks for text fragments with
the addition of a FillTextEmphasisGlyphs StringView/ShapeResult version. 

BUG=714962
R=kojii@chromium.org
==========

to

==========
[LayoutNG] Add NG-specific DrawText support to GraphicsContext

Add LayoutNG versions of the DrawText and DrawEmphasisMarks methods that
operate on TextFragmentPaintInfo instances (wrapping text fragment data)
instead of TextRunPaintInfo instances. Reuses impl when at all possible.

Also adds support for bloberizing emphasis marks for text fragments with
the addition of a FillTextEmphasisGlyphs StringView/ShapeResult version. 

BUG=714962
R=kojii@chromium.org

Review-Url: https://codereview.chromium.org/2845883002
Cr-Commit-Position: refs/heads/master@{#467592}
Committed:
https://chromium.googlesource.com/chromium/src/+/67b9f6e761fba98d3a3e80edec86...
==========

[commit-bot: I haz the power, 2017-04-27 05:33:59]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/67b9f6e761fba98d3a3e80edec86...