Remove RenderObject::requiresForcedStyleRecalcPropagation
authorantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 19 Dec 2016 17:37:11 +0000 (17:37 +0000)
committerantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 19 Dec 2016 17:37:11 +0000 (17:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=166021

Reviewed by Andreas Kling.

It is not called. It also makes no sense architecturally.

* html/shadow/mac/ImageControlsButtonElementMac.cpp:
* html/shadow/mac/ImageControlsRootElementMac.cpp:
* rendering/RenderButton.h:
* rendering/RenderFileUploadControl.h:
* rendering/RenderListItem.h:
* rendering/RenderMedia.h:
* rendering/RenderMenuList.h:
* rendering/RenderMeter.h:
* rendering/RenderObject.h:
(WebCore::RenderObject::requiresForcedStyleRecalcPropagation): Deleted.
* rendering/RenderProgress.h:
* rendering/RenderSlider.h:
* rendering/RenderTextControl.h:
* style/RenderTreeUpdater.cpp:
(WebCore::RenderTreeUpdater::updateElementRenderer):

    This is the only call site. Combination

        update.change == Style::NoChange && parent().styleChange == Style::Force

    is impossible ('Force' inherits) so requiresForcedStyleRecalcPropagation doesn't ever even get called.

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

14 files changed:
Source/WebCore/ChangeLog
Source/WebCore/html/shadow/mac/ImageControlsButtonElementMac.cpp
Source/WebCore/html/shadow/mac/ImageControlsRootElementMac.cpp
Source/WebCore/rendering/RenderButton.h
Source/WebCore/rendering/RenderFileUploadControl.h
Source/WebCore/rendering/RenderListItem.h
Source/WebCore/rendering/RenderMedia.h
Source/WebCore/rendering/RenderMenuList.h
Source/WebCore/rendering/RenderMeter.h
Source/WebCore/rendering/RenderObject.h
Source/WebCore/rendering/RenderProgress.h
Source/WebCore/rendering/RenderSlider.h
Source/WebCore/rendering/RenderTextControl.h
Source/WebCore/style/RenderTreeUpdater.cpp

index cd8056d..47c2c4e 100644 (file)
@@ -1,3 +1,34 @@
+2016-12-19  Antti Koivisto  <antti@apple.com>
+
+        Remove RenderObject::requiresForcedStyleRecalcPropagation
+        https://bugs.webkit.org/show_bug.cgi?id=166021
+
+        Reviewed by Andreas Kling.
+
+        It is not called. It also makes no sense architecturally.
+
+        * html/shadow/mac/ImageControlsButtonElementMac.cpp:
+        * html/shadow/mac/ImageControlsRootElementMac.cpp:
+        * rendering/RenderButton.h:
+        * rendering/RenderFileUploadControl.h:
+        * rendering/RenderListItem.h:
+        * rendering/RenderMedia.h:
+        * rendering/RenderMenuList.h:
+        * rendering/RenderMeter.h:
+        * rendering/RenderObject.h:
+        (WebCore::RenderObject::requiresForcedStyleRecalcPropagation): Deleted.
+        * rendering/RenderProgress.h:
+        * rendering/RenderSlider.h:
+        * rendering/RenderTextControl.h:
+        * style/RenderTreeUpdater.cpp:
+        (WebCore::RenderTreeUpdater::updateElementRenderer):
+
+            This is the only call site. Combination
+
+                update.change == Style::NoChange && parent().styleChange == Style::Force
+
+            is impossible ('Force' inherits) so requiresForcedStyleRecalcPropagation doesn't ever even get called.
+
 2016-12-19  Zan Dobersek  <zdobersek@igalia.com>
 
         [EME] Remove MediaKeySessionType::persistent-usage-record
index 96cd2f5..fd0c3e3 100644 (file)
@@ -53,7 +53,6 @@ private:
     LogicalExtentComputedValues computeLogicalHeight(LayoutUnit logicalHeight, LayoutUnit logicalTop) const override;
 
     const char* renderName() const override { return "RenderImageControlsButton"; }
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
 };
 
 RenderImageControlsButton::RenderImageControlsButton(HTMLElement& element, RenderStyle&& style)
index 3aba922..cf2cdb5 100644 (file)
@@ -46,7 +46,6 @@ private:
     LogicalExtentComputedValues computeLogicalHeight(LayoutUnit logicalHeight, LayoutUnit logicalTop) const override;
 
     const char* renderName() const override { return "RenderImageControls"; }
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
 };
 
 RenderImageControls::RenderImageControls(HTMLElement& element, RenderStyle&& style)
index 0eadd00..96bfda8 100644 (file)
@@ -71,8 +71,6 @@ private:
 
     bool hasLineIfEmpty() const override;
 
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
-
     bool isFlexibleBoxImpl() const override { return true; }
 
     RenderTextFragment* m_buttonText;
index 43c0b94..910a8a0 100644 (file)
@@ -52,8 +52,6 @@ private:
     void computePreferredLogicalWidths() override;
     void paintObject(PaintInfo&, const LayoutPoint&) override;
 
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
-
     int maxFilenameWidth() const;
     
     VisiblePosition positionForPoint(const LayoutPoint&, const RenderRegion*) override;
index 2ac2dce..5176a42 100644 (file)
@@ -73,8 +73,6 @@ private:
 
     void styleDidChange(StyleDifference, const RenderStyle* oldStyle) override;
 
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
-
     void addOverflowFromChildren() override;
     void computePreferredLogicalWidths() override;
 
index 7457d48..7c5cfda 100644 (file)
@@ -53,8 +53,6 @@ private:
     bool isImage() const final { return false; }
     void paintReplaced(PaintInfo&, const LayoutPoint&) override;
 
-    bool requiresForcedStyleRecalcPropagation() const final { return true; }
-
     bool shadowControlsNeedCustomLayoutMetrics() const override { return true; }
     void styleDidChange(StyleDifference, const RenderStyle* oldStyle) final;
 };
index 6e0ba75..815188f 100644 (file)
@@ -81,8 +81,6 @@ private:
 
     void styleDidChange(StyleDifference, const RenderStyle* oldStyle) override;
 
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
-
     // PopupMenuClient methods
     void valueChanged(unsigned listIndex, bool fireOnChange = true) override;
     void selectionChanged(unsigned, bool) override { }
index da95444..92e0a5c 100644 (file)
@@ -43,7 +43,6 @@ private:
 
     const char* renderName() const override { return "RenderMeter"; }
     bool isMeter() const override { return true; }
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
 };
 
 } // namespace WebCore
index 1a67271..f8b4c40 100644 (file)
@@ -327,8 +327,6 @@ public:
     FlowThreadState flowThreadState() const { return m_bitfields.flowThreadState(); }
     void setFlowThreadState(FlowThreadState state) { m_bitfields.setFlowThreadState(state); }
 
-    virtual bool requiresForcedStyleRecalcPropagation() const { return false; }
-
 #if ENABLE(MATHML)
     virtual bool isRenderMathMLBlock() const { return false; }
     virtual bool isRenderMathMLTable() const { return false; }
index 03122d2..3a6d84d 100644 (file)
@@ -43,7 +43,6 @@ public:
 private:
     const char* renderName() const override { return "RenderProgress"; }
     bool isProgress() const override { return true; }
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
     LogicalExtentComputedValues computeLogicalHeight(LayoutUnit logicalHeight, LayoutUnit logicalTop) const override;
 
     void animationTimerFired();
index 535588e..038c840 100644 (file)
@@ -45,7 +45,6 @@ private:
     int baselinePosition(FontBaseline, bool firstLine, LineDirectionMode, LinePositionMode = PositionOnContainingLine) const override;
     void computeIntrinsicLogicalWidths(LayoutUnit& minLogicalWidth, LayoutUnit& maxLogicalWidth) const override;
     void computePreferredLogicalWidths() override;
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
     void layout() override;
 
     bool isFlexibleBoxImpl() const override { return true; }
index 7ab2a47..4176a2c 100644 (file)
@@ -80,8 +80,6 @@ private:
     void addFocusRingRects(Vector<LayoutRect>&, const LayoutPoint& additionalOffset, const RenderLayerModelObject* paintContainer = 0) override;
 
     bool canBeProgramaticallyScrolled() const override { return true; }
-
-    bool requiresForcedStyleRecalcPropagation() const override { return true; }
 };
 
 // Renderer for our inner container, for <search> and others.
index 551cc24..d1a648d 100644 (file)
@@ -283,7 +283,7 @@ void RenderTreeUpdater::updateElementRenderer(Element& element, const Style::Ele
     }
 
     if (update.change == Style::NoChange) {
-        if (pseudoStyleCacheIsInvalid(&renderer, update.style.get()) || (parent().styleChange == Style::Force && renderer.requiresForcedStyleRecalcPropagation())) {
+        if (pseudoStyleCacheIsInvalid(&renderer, update.style.get())) {
             renderer.setStyle(RenderStyle::clone(*update.style), StyleDifferenceEqual);
             return;
         }