BidiResolver should not try to reverse runs when the run count is zero

If m_runs is empty we do not need to call reverseRuns. Additionally, verifies if we have any runs in BidiRunList before trying to reverse.

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

[eseidel, 2013-10-02 17:28:10]

lgtm

https://codereview.chromium.org/25373008/diff/6001/Source/core/platform/text/...
File Source/core/platform/text/BidiResolver.h (left):

https://codereview.chromium.org/25373008/diff/6001/Source/core/platform/text/...
Source/core/platform/text/BidiResolver.h:578: m_runs.reverseRuns(0,
m_runs.runCount() - 1);
Do we ever call this w/o the full length?  If not we could just remove the
parameters.

[igoroliveira, 2013-10-02 17:33:05]

On 2013/10/02 17:28:10, eseidel wrote:
> lgtm
> 
>
https://codereview.chromium.org/25373008/diff/6001/Source/core/platform/text/...
> File Source/core/platform/text/BidiResolver.h (left):
> 
>
https://codereview.chromium.org/25373008/diff/6001/Source/core/platform/text/...
> Source/core/platform/text/BidiResolver.h:578: m_runs.reverseRuns(0,
> m_runs.runCount() - 1);
> Do we ever call this w/o the full length?  If not we could just remove the
> parameters.

BidiResolver<Iterator, Run>::reorderRunsFromLevels calls reverseRuns w/o the
full length.

[commit-bot: I haz the power, 2013-10-02 18:00:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/igor.o@sisa.samsung.com/25373008/10001

[commit-bot: I haz the power, 2013-10-02 21:20:42]

Change committed as 158752