Ignore pos argument if the first render object is not the current one.

Ignore pos argument if the first render object is not the current one.

The pos argument defines a starting position inside the current render
object. However, we might need to use another render object to
determine the text direction. The pos position is invalid from
the perspecitve of the other render object, so we pass 0 instead.

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

[zherczeg, 2014-11-20 11:12:43]

zherczeg.u-szeged@partner.samsung.com changed reviewers:
+ abarth@chromium.org, jchaffraix+rendering@chromium.org, schenney@chromium.org

[zherczeg, 2014-11-20 11:12:43]

Please review this patch.

[leviw_travelin_and_unemployed, 2014-11-20 23:34:44]

leviw@chromium.org changed reviewers:
+ leviw@chromium.org

[leviw_travelin_and_unemployed, 2014-11-20 23:34:44]

lgtm.

https://codereview.chromium.org/737333002/diff/1/Source/core/rendering/BidiRu...
File Source/core/rendering/BidiRunForLine.cpp (right):

https://codereview.chromium.org/737333002/diff/1/Source/core/rendering/BidiRu...
Source/core/rendering/BidiRunForLine.cpp:86: RenderObject* firstRenderObject =
Don't wrap this line.

[zherczeg, 2014-11-21 08:04:02]

The CQ bit was checked by zherczeg.u-szeged@partner.samsung.com

[commit-bot: I haz the power, 2014-11-21 08:04:23]

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

[commit-bot: I haz the power, 2014-11-21 09:19:06]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=185778