Changeset 219121 in webkit

Timestamp:

Jul 4, 2017, 10:58:36 AM (9 years ago)

Author:

Antti Koivisto

Message:

FrameView should not set RenderView::logicalWidth directly for printing
​https://bugs.webkit.org/show_bug.cgi?id=174135

Reviewed by Zalan Bujtas.

Source/WebCore:

Renderer logicalWidth should be set by layout. Direct override by RenderView when printing means
that we don't layout children in all cases when the width changes. This is currently mostly hidden
by spurious layouts but causes problems when trying to fix other things that reduces those.

• page/FrameView.cpp:

(WebCore::FrameView::forceLayoutForPagination):

Instead of calling setLogicalWidth directly call the new setPageLogicalSize that sets both the width
and the height uniformly.

• rendering/RenderView.cpp:

(WebCore::RenderView::updateLogicalWidth):

Use pageLogicalSize->width() in printing state instead of skipping the logical width update entirely.
This ensures that the layout will progress to children when the page logical width changes.

(WebCore::RenderView::initializeLayoutState):
(WebCore::RenderView::layout):
(WebCore::RenderView::pageOrViewLogicalHeight):
(WebCore::RenderView::setPageLogicalSize):

• rendering/RenderView.h:

Replace the existing m_pageLogicalHeight with std::optional m_pageLogicalSize.

LayoutTests:

• platform/mac/printing/width-overflow-expected.txt:

This is a progression, view and root element sizes now match.
Printing output is not affected.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/platform/mac/printing/width-overflow-expected.txt (modified) (1 diff)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/page/FrameView.cpp (modified) (2 diffs)
• Source/WebCore/rendering/RenderView.cpp (modified) (5 diffs)
• Source/WebCore/rendering/RenderView.h (modified) (2 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2017-07-04  Antti Koivisto  <antti@apple.com>
+
+        FrameView should not set RenderView::logicalWidth directly for printing
+        https://bugs.webkit.org/show_bug.cgi?id=174135
+
+        Reviewed by Zalan Bujtas.
+
+        * platform/mac/printing/width-overflow-expected.txt:
+
+            This is a progression, view and root element sizes now match.
+            Printing output is not affected.
+
 2017-07-03  Saam Barati  <sbarati@apple.com>
 

trunk/LayoutTests/platform/mac/printing/width-overflow-expected.txt

@@ -1 +1 @@
 layer at (0,0) size 1300x2284
   RenderView at (0,0) size 1300x2284
-layer at (0,0) size 981x2284
-  RenderBlock {HTML} at (0,0) size 981x2284
-    RenderBody {BODY} at (0,16) size 981x2252
-      RenderBlock {P} at (0,0) size 981x18
+layer at (0,0) size 1300x2284
+  RenderBlock {HTML} at (0,0) size 1300x2284
+    RenderBody {BODY} at (0,16) size 1300x2252
+      RenderBlock {P} at (0,0) size 1300x18
         RenderText {#text} at (0,0) size 765x18
           text run at (0,0) width 765: "To run this test manually, print this page. If the right side of any lines is printed without being truncated, the test passes."

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-07-04  Antti Koivisto  <antti@apple.com>
+
+        FrameView should not set RenderView::logicalWidth directly for printing
+        https://bugs.webkit.org/show_bug.cgi?id=174135
+
+        Reviewed by Zalan Bujtas.
+
+        Renderer logicalWidth should be set by layout. Direct override by RenderView when printing means
+        that we don't layout children in all cases when the width changes. This is currently mostly hidden
+        by spurious layouts but causes problems when trying to fix other things that reduces those.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::forceLayoutForPagination):
+
+            Instead of calling setLogicalWidth directly call the new setPageLogicalSize that sets both the width
+            and the height uniformly.
+
+        * rendering/RenderView.cpp:
+        (WebCore::RenderView::updateLogicalWidth):
+
+            Use pageLogicalSize->width() in printing state instead of skipping the logical width update entirely.
+            This ensures that the layout will progress to children when the page logical width changes.
+
+        (WebCore::RenderView::initializeLayoutState):
+        (WebCore::RenderView::layout):
+        (WebCore::RenderView::pageOrViewLogicalHeight):
+        (WebCore::RenderView::setPageLogicalSize):
+        * rendering/RenderView.h:
+
+            Replace the existing m_pageLogicalHeight with std::optional m_pageLogicalSize.
+
 2017-07-04  Xabier Rodriguez Calvar  <calvaris@igalia.com>
 

trunk/Source/WebCore/page/FrameView.cpp

@@ -4677 +4677 @@
         float pageLogicalHeight = renderView->style().isHorizontalWritingMode() ? pageSize.height() : pageSize.width();
 
-        renderView->setLogicalWidth(floor(pageLogicalWidth));
-        renderView->setPageLogicalHeight(floor(pageLogicalHeight));
+        renderView->setPageLogicalSize({ floor(pageLogicalWidth), floor(pageLogicalHeight) });
         renderView->setNeedsLayoutAndPrefWidthsRecalc();
         forceLayout();
@@ -4696 +4695 @@
             pageLogicalHeight = horizontalWritingMode ? maxPageSize.height() : maxPageSize.width();
 
-            renderView->setLogicalWidth(floor(pageLogicalWidth));
-            renderView->setPageLogicalHeight(floor(pageLogicalHeight));
+            renderView->setPageLogicalSize({ floor(pageLogicalWidth), floor(pageLogicalHeight) });
             renderView->setNeedsLayoutAndPrefWidthsRecalc();
             forceLayout();

trunk/Source/WebCore/rendering/RenderView.cpp

@@ -218 +218 @@
 void RenderView::updateLogicalWidth()
 {
-    if (!shouldUsePrintingLayout())
-        setLogicalWidth(viewLogicalWidth());
+    setLogicalWidth(shouldUsePrintingLayout() ? m_pageLogicalSize->width() : LayoutUnit(viewLogicalWidth()));
 }
 
@@ -269 +268 @@
     state.m_clipped = false;
 
-    state.m_pageLogicalHeight = m_pageLogicalHeight;
+    state.m_pageLogicalHeight = m_pageLogicalSize ? m_pageLogicalSize->height() : LayoutUnit(0);
     state.m_pageLogicalHeightChanged = m_pageLogicalHeightChanged;
     ASSERT(state.m_pageLogicalHeight >= 0);
@@ -340 +339 @@
     StackStats::LayoutCheckPoint layoutCheckPoint;
     if (!document().paginated())
-        setPageLogicalHeight(0);
-
-    if (shouldUsePrintingLayout())
-        m_minPreferredLogicalWidth = m_maxPreferredLogicalWidth = logicalWidth();
+        m_pageLogicalSize = { };
+
+    if (shouldUsePrintingLayout()) {
+        ASSERT(m_pageLogicalSize);
+        m_minPreferredLogicalWidth = m_pageLogicalSize->width();
+        m_maxPreferredLogicalWidth = m_minPreferredLogicalWidth;
+    }
 
     // Use calcWidth/Height to get the new width/height, since this will take the full page zoom factor into account.
@@ -387 +389 @@
 {
     if (document().printing())
-        return pageLogicalHeight();
+        return m_pageLogicalSize->height();
     
     if (multiColumnFlowThread() && !style().hasInlineColumnAxis()) {
@@ -1220 +1222 @@
 }
 
+void RenderView::setPageLogicalSize(LayoutSize size)
+{
+    if (!m_pageLogicalSize || m_pageLogicalSize->height() != size.height())
+        m_pageLogicalHeightChanged = true;
+
+    m_pageLogicalSize = size;
+}
+
 float RenderView::zoomFactor() const
 {

trunk/Source/WebCore/rendering/RenderView.h

@@ -142 +142 @@
     void updateHitTestResult(HitTestResult&, const LayoutPoint&) override;
 
-    LayoutUnit pageLogicalHeight() const { return m_pageLogicalHeight; }
-    void setPageLogicalHeight(LayoutUnit height)
-    {
-        if (m_pageLogicalHeight != height) {
-            m_pageLogicalHeight = height;
-            m_pageLogicalHeightChanged = true;
-        }
-    }
+    void setPageLogicalSize(LayoutSize);
     LayoutUnit pageOrViewLogicalHeight() const;
 
@@ -372 +365 @@
 
     std::unique_ptr<ImageQualityController> m_imageQualityController;
-    LayoutUnit m_pageLogicalHeight;
+    std::optional<LayoutSize> m_pageLogicalSize;
     bool m_pageLogicalHeightChanged { false };
     std::unique_ptr<LayoutState> m_layoutState;