[LFC] FormattingContext base class should not declare computeStaticPosition.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Nov 2018 14:58:16 +0000 (14:58 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Nov 2018 14:58:16 +0000 (14:58 +0000)
https://bugs.webkit.org/show_bug.cgi?id=191683

Reviewed by Antti Koivisto.

Apparently only BlockFormattingContext uses it.

* layout/FormattingContext.h:
* layout/blockformatting/BlockFormattingContext.h:
* layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::computeStaticPosition const): Deleted.
* layout/inlineformatting/InlineFormattingContext.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@238226 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/layout/FormattingContext.h
Source/WebCore/layout/blockformatting/BlockFormattingContext.h
Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp
Source/WebCore/layout/inlineformatting/InlineFormattingContext.h

index 041d376..ea0c592 100644 (file)
@@ -1,3 +1,18 @@
+2018-11-15  Zalan Bujtas  <zalan@apple.com>
+
+        [LFC] FormattingContext base class should not declare computeStaticPosition.
+        https://bugs.webkit.org/show_bug.cgi?id=191683
+
+        Reviewed by Antti Koivisto.
+
+        Apparently only BlockFormattingContext uses it.
+
+        * layout/FormattingContext.h:
+        * layout/blockformatting/BlockFormattingContext.h:
+        * layout/inlineformatting/InlineFormattingContext.cpp:
+        (WebCore::Layout::InlineFormattingContext::computeStaticPosition const): Deleted.
+        * layout/inlineformatting/InlineFormattingContext.h:
+
 2018-11-14  Zalan Bujtas  <zalan@apple.com>
 
         [LFC][IFC] Add support for in-flow positioned inline boxes.
index 6769a69..a4145a4 100644 (file)
@@ -69,8 +69,6 @@ protected:
     LayoutState& layoutState() const;
     const Box& root() const { return *m_root; }
 
-    virtual void computeStaticPosition(const Box&) const = 0;
-
     void computeBorderAndPadding(const Box&) const;
 
 #ifndef NDEBUG
index 9a9a0c6..f61a756 100644 (file)
@@ -56,7 +56,7 @@ private:
     void computeWidthAndMargin(const Box&) const;
     void computeHeightAndMargin(const Box&) const;
 
-    void computeStaticPosition(const Box&) const override;
+    void computeStaticPosition(const Box&) const;
     void computeFloatingPosition(const FloatingContext&, const Box&) const;
     void computePositionToAvoidFloats(const FloatingContext&, const Box&) const;
     void computeVerticalPositionForFloatClear(const FloatingContext&, const Box&) const;
index 5734420..9ee7390 100644 (file)
@@ -442,10 +442,6 @@ void InlineFormattingContext::placeInFlowPositionedChildren(Line::RunRange runRa
     }
 }
 
-void InlineFormattingContext::computeStaticPosition(const Box&) const
-{
-}
-
 void InlineFormattingContext::collectInlineContentForSubtree(const Box& root, InlineRunProvider& inlineRunProvider) const
 {
     // Collect inline content recursively and set breaking rules for the inline elements (for paddings, margins, positioned element etc).
index fe69a7a..4aa8434 100644 (file)
@@ -120,7 +120,6 @@ private:
     void computeHeightAndMargin(const Box&) const;
     void computeWidthAndMargin(const Box&) const;
     void computeFloatPosition(const FloatingContext&, Line&, const Box&) const;
-    void computeStaticPosition(const Box&) const override;
     void placeInFlowPositionedChildren(Line::RunRange) const;
 
     void collectInlineContent(InlineRunProvider&) const;