Add missing reset calls after continue in layoutRunsAndFloatsInRange().

Source/core/rendering/RenderBlockLineLayout.cpp

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2003, 2004, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All right reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 862 matching lines @@
                     if (adjustment) {
                         LayoutUnit oldLineWidth = availableLogicalWidthForLine(oldLogicalHeight, layoutState.lineInfo().isFirstLine());
                         lineBox->adjustBlockDirectionPosition(adjustment.toFloat());
                         if (layoutState.usesPaintInvalidationBounds())
                             layoutState.updatePaintInvalidationRangeFromBox(lineBox);
 
                         if (availableLogicalWidthForLine(oldLogicalHeight + adjustment, layoutState.lineInfo().isFirstLine()) != oldLineWidth) {
                             // We have to delete this line, remove all floats that got added, and let line layout re-run.
                             lineBox->deleteLine();
                             endOfLine = restartLayoutRunsAndFloatsInRange(oldLogicalHeight, oldLogicalHeight + adjustment, lastFloatFromPreviousLine, resolver, previousEndofLine);
+                            lineMidpointState.reset();
+                            resolver.setPosition(endOfLine, numberOfIsolateAncestors(endOfLine));

[rhogan, 2014/11/03 18:58:12]

This looks right to me. The lineMidpointState.reset() looks superfluous but if
you want to remove it you need to look closely at the logic that breaks before
it gets called at the top of the function.

In the bug you note that there's another superfluous-looking call to
lineMidpointState.reset() at the end of the function - that is definitely worth
addressing in a follow-up patch. If I were you I would keep the change above as
it is and address the superfluous calls in a follow-up.

You also need to add a test here. Once you have that you should ping an OWNER
like leviw or jchaffraix.

                             continue;
                         }
 
                         setLogicalHeight(lineBox->lineBottomWithLeading());
                     }
                 }
             }
         }
 
         for (size_t i = 0; i < lineBreaker.positionedObjects().size(); ++i)
@@ skipping 1152 matching lines @@
     float logicalLeft = logicalLeftOffsetForLine(logicalHeight(), false).toFloat();
     float availableLogicalWidth = logicalRightOffsetForLine(logicalHeight(), false) - logicalLeft;
     updateLogicalWidthForAlignment(textAlign, 0, 0, logicalLeft, totalLogicalWidth, availableLogicalWidth, 0);
 
     if (!style()->isLeftToRightDirection())
         return logicalWidth() - logicalLeft;
     return logicalLeft;
 }
 
 }