Fix RenderViewImplTest.GetCompositionCharacterBoundsTest

Fix RenderViewImplTest.GetCompositionCharacterBoundsTest

This test expects OnImeSetComposition() returns width=0 for between
surrogate pairs. In Blink, this calls:
  Editor::firstRectForRange(const EphemeralRange& range)
which generates the character bounds from the caret position of
downstream and upstream.

In this specific case, when the test run with fonts that has more code
point coverage, the line wraps, and Editor adds |extraWidthToEndOfLine|
which confuses the test.

This patch fixes the test by not to wrap the line,

BUG=508747
Committed: https://crrev.com/378d5bc14d6503b04787acfe9df3e18f0acb647b
Cr-Commit-Position: refs/heads/master@{#379086}

[kojii, 2016-03-03 14:23:15]

Description was changed from

==========
GetCompositionCharacterBoundsTest

BUG=
==========

to

==========
Fix RenderViewImplTest.GetCompositionCharacterBoundsTest

This test expects OnImeSetComposition() returns width=0 for between
surrogate pairs. In Blink, this calls:
  Editor::firstRectForRange(const EphemeralRange& range)
which generates the character bounds from the caret position of
downstream and upstream.

In this specific case, when the test run with fonts that has more code
point coverage, the line wraps, and Editor adds |extraWidthToEndOfLine|
which confuses the test.

This patch fixes the test by not to wrap the line,

BUG=508747
==========

[kojii, 2016-03-03 14:23:54]

kojii@chromium.org changed reviewers:
+ nasko@chromium.org

[kojii, 2016-03-03 14:23:55]

PTAL.

[nasko, 2016-03-03 18:09:23]

LGTM

[kojii, 2016-03-03 20:55:14]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2016-03-03 20:55:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1762673002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1762673002/1

[commit-bot: I haz the power, 2016-03-03 21:02:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-03 21:03:52]

Description was changed from

==========
Fix RenderViewImplTest.GetCompositionCharacterBoundsTest

This test expects OnImeSetComposition() returns width=0 for between
surrogate pairs. In Blink, this calls:
  Editor::firstRectForRange(const EphemeralRange& range)
which generates the character bounds from the caret position of
downstream and upstream.

In this specific case, when the test run with fonts that has more code
point coverage, the line wraps, and Editor adds |extraWidthToEndOfLine|
which confuses the test.

This patch fixes the test by not to wrap the line,

BUG=508747
==========

to

==========
Fix RenderViewImplTest.GetCompositionCharacterBoundsTest

This test expects OnImeSetComposition() returns width=0 for between
surrogate pairs. In Blink, this calls:
  Editor::firstRectForRange(const EphemeralRange& range)
which generates the character bounds from the caret position of
downstream and upstream.

In this specific case, when the test run with fonts that has more code
point coverage, the line wraps, and Editor adds |extraWidthToEndOfLine|
which confuses the test.

This patch fixes the test by not to wrap the line,

BUG=508747

Committed: https://crrev.com/378d5bc14d6503b04787acfe9df3e18f0acb647b
Cr-Commit-Position: refs/heads/master@{#379086}
==========

[commit-bot: I haz the power, 2016-03-03 21:03:54]

Patchset 1 (id:??) landed as
https://crrev.com/378d5bc14d6503b04787acfe9df3e18f0acb647b
Cr-Commit-Position: refs/heads/master@{#379086}