Take text path into account when stripping spaces

Take text path into account when stripping spaces

Take the text path (simple vs complex) into account when computing the
width of a space character used to strip leading and trailing spaces.

R=leviw@chromium.org
BUG=427415
TEST=fast/text/international/complex-text-leading-space-wrapping.html

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185148

[leviw_travelin_and_unemployed, 2014-11-11 21:33:42]

lgtm

https://codereview.chromium.org/721433002/diff/1/LayoutTests/fast/text/intern...
File
LayoutTests/fast/text/international/complext-text-leading-space-wrapping-expected.html
(right):

https://codereview.chromium.org/721433002/diff/1/LayoutTests/fast/text/intern...
LayoutTests/fast/text/international/complext-text-leading-space-wrapping-expected.html:19:
<input type="checkbox" name="advanced" id="advanced-box">
nits: are name="advanced" and id="chromiumsync-advanced-box" needed?
Also "complext" in the filename.

[eae, 2014-11-11 21:34:28]

On 2014/11/11 21:33:42, leviw wrote:
> lgtm
> 
>
https://codereview.chromium.org/721433002/diff/1/LayoutTests/fast/text/intern...
> File
>
LayoutTests/fast/text/international/complext-text-leading-space-wrapping-expected.html
> (right):
> 
>
https://codereview.chromium.org/721433002/diff/1/LayoutTests/fast/text/intern...
>
LayoutTests/fast/text/international/complext-text-leading-space-wrapping-expected.html:19:
> <input type="checkbox" name="advanced" id="advanced-box">
> nits: are name="advanced" and id="chromiumsync-advanced-box" needed?
> Also "complext" in the filename.

No.. Doh. Thanks levi.

[eae, 2014-11-11 21:41:42]

Patchset #2 (id:20001) has been deleted

[eae, 2014-11-11 21:41:56]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2014-11-11 21:42:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/721433002/40001

[commit-bot: I haz the power, 2014-11-11 22:45:17]

Committed patchset #2 (id:40001) as 185148