Revert BidiResolver::createBidiRunsForLine changes

Revert BidiResolver::createBidiRunsForLine changes

Revert the following changes (temporarily):
  r179177 e328f38 "Have BidiRunList destructor delete runs"
  r179068 db7714a "Fix loop condition in BidiRunList::replaceRunWithRun"
  r178879 2b6511b "Change BidiResolver::createBidiRunsForLine to take r"

The change to BidiResolver in r178879 caused a bug in the handling of
isolates in certain edge cases that is proving more complicated than
expected to fix. Rather than leaving things partially broken on trunk
this change reverts the change in question. Once I've come up with a
satisfactory solution I plan to reland the changes mentioned above.

R=leviw@chromium.org
BUG=398339
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179342

[leviw_travelin_and_unemployed, 2014-07-31 17:55:20]

lgtm

[commit-bot: I haz the power, 2014-07-31 17:56:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/eae@chromium.org/433923003/1

[commit-bot: I haz the power, 2014-07-31 19:00:44]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/17924)

[commit-bot: I haz the power, 2014-07-31 20:05:12]

Change committed as 179342