Have BidiRunList destructor delete runs

Source/core/rendering/RenderBlockLineLayout.cpp

Index: Source/core/rendering/RenderBlockLineLayout.cpp
diff --git a/Source/core/rendering/RenderBlockLineLayout.cpp b/Source/core/rendering/RenderBlockLineLayout.cpp
index e68107b917c5314768cb6b64f85f0f98dd8a4313..ce2f703ced80ff2532148199abb0823023d308e5 100644
--- a/Source/core/rendering/RenderBlockLineLayout.cpp
+++ b/Source/core/rendering/RenderBlockLineLayout.cpp
@@ -820,7 +820,10 @@ void RenderBlockFlow::layoutRunsAndFloatsInRange(LineLayoutState& layoutState,
         renderTextInfo.m_lineBreakIterator.resetPriorContext();
         if (resolver.position().atEnd()) {
             // FIXME: We shouldn't be creating any runs in nextLineBreak to begin with!
-            // Once BidiRunList is separated from BidiResolver this will not be needed.
+            // Figure out why we create new runs in nextLineBreak and why we
+            // seem to rely on that behavior.
+            // Removing the addRun calls in LineBreaker::skipLeadingWhitespace
+            // breaks a bunch of bidi layout tests.
             bidiRuns.deleteRuns();
             resolver.markCurrentRunEmpty(); // FIXME: This can probably be replaced by an ASSERT (or just removed).
             layoutState.setCheckForFloatsFromLastLine(true);