[webkit-changes] [178964] trunk/Source/WebCore

Thu, 22 Jan 2015 15:43:14 -0800

Title: [178964] trunk/Source/WebCore
Revision
178964
Author
za...@apple.com
Date
2015-01-22 15:42:46 -0800 (Thu, 22 Jan 2015)

Log Message

Simple line layout: Reset LineState when starting a new line.
https://bugs.webkit.org/show_bug.cgi?id=140789

Reviewed by Antti Koivisto.

Reset LineState when starting a new line to ensure we always
start with a clean state.

No change in functionality.

* rendering/SimpleLineLayout.cpp:
(WebCore::SimpleLineLayout::updateLineConstrains):
(WebCore::SimpleLineLayout::initializeNewLine):
(WebCore::SimpleLineLayout::splitFragmentToFitLine):
(WebCore::SimpleLineLayout::createTextRuns):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (178963 => 178964)


--- trunk/Source/WebCore/ChangeLog	2015-01-22 23:35:15 UTC (rev 178963)
+++ trunk/Source/WebCore/ChangeLog	2015-01-22 23:42:46 UTC (rev 178964)
@@ -1,3 +1,21 @@
+2015-01-22  Zalan Bujtas  <za...@apple.com>
+
+        Simple line layout: Reset LineState when starting a new line.
+        https://bugs.webkit.org/show_bug.cgi?id=140789
+
+        Reviewed by Antti Koivisto.
+
+        Reset LineState when starting a new line to ensure we always
+        start with a clean state.
+
+        No change in functionality.
+
+        * rendering/SimpleLineLayout.cpp:
+        (WebCore::SimpleLineLayout::updateLineConstrains):
+        (WebCore::SimpleLineLayout::initializeNewLine):
+        (WebCore::SimpleLineLayout::splitFragmentToFitLine):
+        (WebCore::SimpleLineLayout::createTextRuns):
+
 2015-01-22  Roger Fong  <roger_f...@apple.com>
 
         [WebGL2] Create empty interface files for new WebGL2 objects.

Modified: trunk/Source/WebCore/rendering/SimpleLineLayout.cpp (178963 => 178964)


--- trunk/Source/WebCore/rendering/SimpleLineLayout.cpp	2015-01-22 23:35:15 UTC (rev 178963)
+++ trunk/Source/WebCore/rendering/SimpleLineLayout.cpp	2015-01-22 23:42:46 UTC (rev 178964)
@@ -350,21 +350,35 @@
         ++lineState.position;
 }
 
-static void initializeNewLine(LineState& lineState, const FlowContents& flowContents, unsigned lineStartRunIndex)
+static void updateLineConstrains(const RenderBlockFlow& flow, float& availableWidth, float& logicalLeftOffset)
 {
+    LayoutUnit height = flow.logicalHeight();
+    LayoutUnit logicalHeight = flow.minLineHeightForReplacedRenderer(false, 0);
+    float logicalRightOffset = flow.logicalRightOffsetForLine(height, false, logicalHeight);
+    logicalLeftOffset = flow.logicalLeftOffsetForLine(height, false, logicalHeight);
+    availableWidth = std::max<float>(0, logicalRightOffset - logicalLeftOffset);
+}
+
+static LineState initializeNewLine(const LineState& previousLine, const RenderBlockFlow& flow, const FlowContents& flowContents, unsigned lineStartRunIndex)
+{
+    LineState lineState;
+    lineState.jumpTo(previousLine.position, 0);
     lineState.lineStartRunIndex = lineStartRunIndex;
+    updateLineConstrains(flow, lineState.availableWidth, lineState.logicalLeftOffset);
     // Skip leading whitespace if collapsing whitespace, unless there's an uncommitted fragment pushed from the previous line.
     // FIXME: Be smarter when the run from the previous line does not fit the current line. Right now, we just reprocess it.
-    if (lineState.oveflowedFragment.width) {
-        if (lineState.fits(lineState.oveflowedFragment.width))
-            lineState.addUncommitted(lineState.oveflowedFragment);
-        else
-            lineState.jumpTo(lineState.oveflowedFragment.start, 0); // Start over with this fragment.
+    if (previousLine.oveflowedFragment.width) {
+        if (lineState.fits(previousLine.oveflowedFragment.width))
+            lineState.addUncommitted(previousLine.oveflowedFragment);
+        else {
+            // Start over with this fragment.
+            lineState.jumpTo(previousLine.oveflowedFragment.start, 0);
+        }
     } else {
         unsigned spaceCount = 0;
-        lineState.jumpTo(flowContents.style().collapseWhitespace ? flowContents.findNextNonWhitespacePosition(lineState.position, spaceCount) : lineState.position, 0);
+        lineState.jumpTo(flowContents.style().collapseWhitespace ? flowContents.findNextNonWhitespacePosition(previousLine.position, spaceCount) : previousLine.position, 0);
     }
-    lineState.oveflowedFragment = FlowContents::TextFragment();
+    return lineState;
 }
 
 static FlowContents::TextFragment splitFragmentToFitLine(FlowContents::TextFragment& fragmentToSplit, float availableWidth, bool keepAtLeastOneCharacter, const FlowContents& flowContents)
@@ -401,7 +415,7 @@
         ++right;
     FlowContents::TextFragment fragmentForNextLine(fragmentToSplit);
     fragmentToSplit.end = right;
-    fragmentToSplit.width = fragmentToSplit.isEmpty() ? 0 : flowContents.textWidth(fragmentToSplit.start, right, 0);
+    fragmentToSplit.width = fragmentToSplit.isEmpty() ? 0 : flowContents.textWidth(fragmentToSplit.start, fragmentToSplit.end, 0);
 
     fragmentForNextLine.start = fragmentToSplit.end;
     fragmentForNextLine.width -= fragmentToSplit.width;
@@ -509,15 +523,6 @@
     } while (++runIndex < lineRuns.size());
 }
 
-static void updateLineConstrains(const RenderBlockFlow& flow, float& availableWidth, float& logicalLeftOffset)
-{
-    LayoutUnit height = flow.logicalHeight();
-    LayoutUnit logicalHeight = flow.minLineHeightForReplacedRenderer(false, 0);
-    float logicalRightOffset = flow.logicalRightOffsetForLine(height, false, logicalHeight);
-    logicalLeftOffset = flow.logicalLeftOffsetForLine(height, false, logicalHeight);
-    availableWidth = std::max<float>(0, logicalRightOffset - logicalLeftOffset);
-}
-
 static void createTextRuns(Layout::RunVector& runs, RenderBlockFlow& flow, unsigned& lineCount)
 {
     LayoutUnit borderAndPaddingBefore = flow.borderAndPaddingBefore();
@@ -528,8 +533,7 @@
 
     do {
         flow.setLogicalHeight(lineHeight * lineCount + borderAndPaddingBefore);
-        updateLineConstrains(flow, lineState.availableWidth, lineState.logicalLeftOffset);
-        initializeNewLine(lineState, flowContents, runs.size());
+        lineState = initializeNewLine(lineState, flow, flowContents, runs.size());
         isEndOfContent = createLineRuns(lineState, runs, flowContents);
         closeLineEndingAndAdjustRuns(lineState, runs, lineCount, flowContents);
     } while (!isEndOfContent);

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to