Remove unused Element::alwaysCreateUserAgentShadowRoot()
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Nov 2017 23:32:14 +0000 (23:32 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Nov 2017 23:32:14 +0000 (23:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=179141

Reviewed by Ryosuke Niwa.

In changeset r208135 Element::createShadowRoot() was removed. And Element::createShadowRoot()
was the only caller of Element::alwaysCreateUserAgentShadowRoot(). We should remove
Element::alwaysCreateUserAgentShadowRoot() as it is unused.

* dom/Element.h:
(WebCore::Element::didAddUserAgentShadowRoot):
(WebCore::Element::alwaysCreateUserAgentShadowRoot const): Deleted.
* html/HTMLButtonElement.h:
* html/HTMLFormControlElement.h:
* html/HTMLMediaElement.h:

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

Source/WebCore/ChangeLog
Source/WebCore/dom/Element.h
Source/WebCore/html/HTMLButtonElement.h
Source/WebCore/html/HTMLFormControlElement.h
Source/WebCore/html/HTMLMediaElement.h

index daf6c5f..59e56d7 100644 (file)
@@ -1,3 +1,21 @@
+2017-11-01  Daniel Bates  <dabates@apple.com>
+
+        Remove unused Element::alwaysCreateUserAgentShadowRoot()
+        https://bugs.webkit.org/show_bug.cgi?id=179141
+
+        Reviewed by Ryosuke Niwa.
+
+        In changeset r208135 Element::createShadowRoot() was removed. And Element::createShadowRoot()
+        was the only caller of Element::alwaysCreateUserAgentShadowRoot(). We should remove
+        Element::alwaysCreateUserAgentShadowRoot() as it is unused.
+
+        * dom/Element.h:
+        (WebCore::Element::didAddUserAgentShadowRoot):
+        (WebCore::Element::alwaysCreateUserAgentShadowRoot const): Deleted.
+        * html/HTMLButtonElement.h:
+        * html/HTMLFormControlElement.h:
+        * html/HTMLMediaElement.h:
+
 2017-11-01  Chris Dumez  <cdumez@apple.com>
 
         Drop running Service Worker Jobs on a background thread
index b06f19b..87427fb 100644 (file)
@@ -580,7 +580,6 @@ private:
     bool isUserActionElementHovered() const;
 
     virtual void didAddUserAgentShadowRoot(ShadowRoot*) { }
-    virtual bool alwaysCreateUserAgentShadowRoot() const { return false; }
 
     // FIXME: Remove the need for Attr to call willModifyAttribute/didModifyAttribute.
     friend class Attr;
index fb1661b..2ec8a4e 100644 (file)
@@ -50,9 +50,6 @@ private:
 
     RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) final;
 
-    // HTMLFormControlElement always creates one, but buttons don't need it.
-    bool alwaysCreateUserAgentShadowRoot() const final { return false; }
-
     void parseAttribute(const QualifiedName&, const AtomicString&) final;
     bool isPresentationAttribute(const QualifiedName&) const final;
     void defaultEventHandler(Event&) final;
index 6fb1b15..3c28d27 100644 (file)
@@ -166,7 +166,6 @@ private:
     bool matchesInvalidPseudoClass() const override;
 
     bool isFormControlElement() const final { return true; }
-    bool alwaysCreateUserAgentShadowRoot() const override { return true; }
 
     int tabIndex() const final;
 
index 76feabd..62edbc2 100644 (file)
@@ -587,8 +587,6 @@ protected:
 private:
     void createMediaPlayer();
 
-    bool alwaysCreateUserAgentShadowRoot() const override { return true; }
-
     bool supportsFocus() const override;
     bool isMouseFocusable() const override;
     bool rendererIsNeeded(const RenderStyle&) override;