Word spacing in RTL for segments

Word spacing in RTL for segments

Removed m_stop and length comparison as they carry different units i.e. if there are two char text then m_stop will be 1 and length 2. As do not compare, needWordSpace does not set correct value.  

Also by default condition for needSpacing has been changed to true because if first character is space it was getting omitted. 

BUG=344873
R=leviw, eae
TEST=Word spacing in RTL with different segments handling.

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

[Habib Virji, 2014-06-17 18:39:28]

Patch#1, include changes for resolving all issues raised in the bug. Patch#2 is
simplified version of the patch without bidi changes.

Bidi changes are required as BidiRun is not created when space and neutral
character(@) is present. It relies on l1 rule to be applied. But l1 only check
if space exists after the text, not before it. Since I could not find, a good
way of detecting whitespace I have rolled out change as it fails other test.

[eae, 2014-06-17 19:09:08]

LGTM++!

[behdad_google, 2014-06-17 19:32:59]

lgtm

[Habib Virji, 2014-06-17 22:27:29]

The CQ bit was checked by habib.virji@samsung.com

[commit-bot: I haz the power, 2014-06-17 22:28:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/habib.virji@samsung.com/343443002/20001

[commit-bot: I haz the power, 2014-06-18 02:13:36]

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

[commit-bot: I haz the power, 2014-06-18 02:13:37]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/12412)

[Habib Virji, 2014-06-18 11:02:27]

The CQ bit was checked by habib.virji@samsung.com

[commit-bot: I haz the power, 2014-06-18 11:02:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/habib.virji@samsung.com/343443002/80001

[commit-bot: I haz the power, 2014-06-18 13:27:11]

Change committed as 176416