Bidi-Isolate inlines break layout with collapsed whitespace

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 8e4c88471ce06321619fd50c1e40465271b38dbf..470e88a8bd90f6e9e0c76ff78008b39d90e7f98b 100644
--- a/Source/core/rendering/RenderBlockLineLayout.cpp
+++ b/Source/core/rendering/RenderBlockLineLayout.cpp
@@ -1303,6 +1303,8 @@ static inline void constructBidiRunsForSegment(InlineBidiResolver& topResolver,
         ASSERT(isolatedInline);
 
         InlineBidiResolver isolatedResolver;
+        LineMidpointState& isolatedLineMidpointState = isolatedResolver.midpointState();
+        isolatedLineMidpointState = topResolver.midpointStateForIsolatedRun(isolatedRun);
         EUnicodeBidi unicodeBidi = isolatedInline->style()->unicodeBidi();
         TextDirection direction = isolatedInline->style()->direction();
         if (unicodeBidi == Plaintext) {
@@ -1339,8 +1341,12 @@ static inline void constructBidiRunsForSegment(InlineBidiResolver& topResolver,
         // to the top resolver's list for later processing.
         if (!isolatedResolver.isolatedRuns().isEmpty()) {
             topResolver.isolatedRuns().append(isolatedResolver.isolatedRuns());
-            isolatedResolver.isolatedRuns().clear();
             currentRoot = isolatedInline;
+            while (!isolatedResolver.isolatedRuns().isEmpty()) {

[leviw_travelin_and_unemployed, 2013/10/07 21:58:42]

It's weird to have nested while loops with the same conditional. If this is
really necessary, can you put it in a function with a descriptive name?

+                BidiRun* run = isolatedResolver.isolatedRuns().last();
+                isolatedResolver.isolatedRuns().removeLast();
+                topResolver.setMidpointStateForIsolatedRun(run, isolatedResolver.midpointStateForIsolatedRun(run));
+            }
         }
     }
 }