Emphasis glyphBuffer to ignore ellipsis when direction is RTL

Avoid drawing emphasis glyphs over ellipsis marks

Accumulate initial advance instead of using "0" non-drawing glyphs.

BUG=408544
R=dominik.rottsches@intel.com, behdad
TEST=fast/text/emphasis-ellipsis-complextext.html

[Habib Virji, 2014-11-24 16:47:33]

@Dominik: It is based on previous CL you reviewed:
https://codereview.chromium.org/613193002/. 

I have used advancedSoFar, to store omitted glyphs. No empty glyph are added.
Tests too are now covering both RTL and LTR.

Moved to new CL as old CL is bit old and git cl patch, does not work because
code was using old file path.

[Dominik Röttsches, 2014-11-24 17:04:56]

Thanks for the update and incorporating the review comments, looks much better.
LGTM.

[Habib Virji, 2014-11-24 17:08:25]

On 2014/11/24 17:04:56, Dominik Röttsches wrote:
> Thanks for the update and incorporating the review comments, looks much
better.
> LGTM.

Thanks a lot for your patience in reviewing and sorry for all hassles.

[Dominik Röttsches, 2014-11-24 17:11:34]

One thing: Before committing, could you please update the description to be more
meaningful and correct:

"Avoid drawing emphasis glyphs over ellipsis marks

Accumulate initial advance instead of using "0" non-drawing glyphs.

BUG=408544
R=dominik.rottsches@intel.com, behdad
TEST=fast/text/emphasis-ellipsis-complextext.html"

[mmenke, 2014-11-24 17:14:24]

The CQ bit was checked by mmenke@chromium.org

[mmenke, 2014-11-24 17:14:51]

The CQ bit was unchecked by mmenke@chromium.org

[Habib Virji, 2014-11-24 17:15:46]

On 2014/11/24 17:11:34, Dominik Röttsches wrote:
> One thing: Before committing, could you please update the description to be
more
> meaningful and correct:
> 
> "Avoid drawing emphasis glyphs over ellipsis marks
> 
> Accumulate initial advance instead of using "0" non-drawing glyphs.
> 
> BUG=408544
> mailto:R=dominik.rottsches@intel.com, behdad
> TEST=fast/text/emphasis-ellipsis-complextext.html"

Thanks. Updated.