Move hasAuthorShadowRoot to Element
authoresprehn@chromium.org <esprehn@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 28 Jan 2013 21:45:11 +0000 (21:45 +0000)
committeresprehn@chromium.org <esprehn@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 28 Jan 2013 21:45:11 +0000 (21:45 +0000)
https://bugs.webkit.org/show_bug.cgi?id=108071

Reviewed by Dimitri Glazkov.

Move hasAuthorShadowRoot to Element and get rid of unneccesary booleans
in HTMLProgressElement and HTMLMeterElement. Also get rid of
ShadowRoot::isAccessible since it obfuscates what's actually happening
inside of Element::shadowRoot().

No new tests, just refactoring.

* dom/Element.cpp:
(WebCore::Element::shadowRoot):
(WebCore::Element::hasAuthorShadowRoot):
* dom/Element.h:
(Element):
* dom/ShadowRoot.h:
* html/HTMLMeterElement.cpp:
(WebCore::HTMLMeterElement::HTMLMeterElement):
* html/HTMLMeterElement.h:
(HTMLMeterElement):
* html/HTMLProgressElement.cpp:
(WebCore::HTMLProgressElement::HTMLProgressElement):
* html/HTMLProgressElement.h:
(HTMLProgressElement):

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

Source/WebCore/ChangeLog
Source/WebCore/dom/Element.cpp
Source/WebCore/dom/Element.h
Source/WebCore/dom/ShadowRoot.h
Source/WebCore/html/HTMLMeterElement.cpp
Source/WebCore/html/HTMLMeterElement.h
Source/WebCore/html/HTMLProgressElement.cpp
Source/WebCore/html/HTMLProgressElement.h

index 07d9f4c..ef689bd 100644 (file)
@@ -1,5 +1,34 @@
 2013-01-28  Elliott Sprehn  <esprehn@chromium.org>
 
+        Move hasAuthorShadowRoot to Element
+        https://bugs.webkit.org/show_bug.cgi?id=108071
+
+        Reviewed by Dimitri Glazkov.
+
+        Move hasAuthorShadowRoot to Element and get rid of unneccesary booleans
+        in HTMLProgressElement and HTMLMeterElement. Also get rid of
+        ShadowRoot::isAccessible since it obfuscates what's actually happening
+        inside of Element::shadowRoot().
+
+        No new tests, just refactoring.
+
+        * dom/Element.cpp:
+        (WebCore::Element::shadowRoot):
+        (WebCore::Element::hasAuthorShadowRoot):
+        * dom/Element.h:
+        (Element):
+        * dom/ShadowRoot.h:
+        * html/HTMLMeterElement.cpp:
+        (WebCore::HTMLMeterElement::HTMLMeterElement):
+        * html/HTMLMeterElement.h:
+        (HTMLMeterElement):
+        * html/HTMLProgressElement.cpp:
+        (WebCore::HTMLProgressElement::HTMLProgressElement):
+        * html/HTMLProgressElement.h:
+        (HTMLProgressElement):
+
+2013-01-28  Elliott Sprehn  <esprehn@chromium.org>
+
         Move ensureUserAgentShadowRoot to Element
         https://bugs.webkit.org/show_bug.cgi?id=108070
 
index 204b707..72f602c 100644 (file)
@@ -1471,9 +1471,9 @@ ShadowRoot* Element::shadowRoot() const
     if (!elementShadow)
         return 0;
     ShadowRoot* shadowRoot = elementShadow->youngestShadowRoot();
-    if (!shadowRoot->isAccessible())
-        return 0;
-    return shadowRoot;
+    if (shadowRoot->type() == ShadowRoot::AuthorShadowRoot)
+        return shadowRoot;
+    return 0;
 }
 
 ShadowRoot* Element::userAgentShadowRoot() const
index e953d27..e071222 100644 (file)
@@ -293,6 +293,7 @@ public:
     PassRefPtr<ShadowRoot> createShadowRoot(ExceptionCode&);
     ShadowRoot* shadowRoot() const;
 
+    bool hasAuthorShadowRoot() const { return shadowRoot(); }
     virtual void willAddAuthorShadowRoot() { }
     virtual bool areAuthorShadowsAllowed() const { return true; }
 
index 1526938..83e9bf3 100644 (file)
@@ -93,7 +93,6 @@ public:
     ScopeContentDistribution* ensureScopeDistribution();
 
     ShadowRootType type() const { return m_isAuthorShadowRoot ? AuthorShadowRoot : UserAgentShadowRoot; }
-    bool isAccessible() const { return type() == AuthorShadowRoot; }
 
     PassRefPtr<Node> cloneNode(bool, ExceptionCode&);
 
index f59d79e..465d0ff 100644 (file)
@@ -43,7 +43,6 @@ using namespace HTMLNames;
 
 HTMLMeterElement::HTMLMeterElement(const QualifiedName& tagName, Document* document)
     : LabelableElement(tagName, document)
-    , m_hasAuthorShadowRoot(false)
 {
     ASSERT(hasTagName(meterTag));
 }
@@ -225,11 +224,6 @@ void HTMLMeterElement::didElementStateChange()
         render->updateFromElement();
 }
 
-void HTMLMeterElement::willAddAuthorShadowRoot()
-{
-    m_hasAuthorShadowRoot = true;
-}
-
 RenderMeter* HTMLMeterElement::renderMeter() const
 {
     if (renderer() && renderer()->isMeter())
index d1107a4..f58e351 100644 (file)
@@ -39,8 +39,6 @@ public:
         GaugeRegionEvenLessGood
     };
 
-    bool hasAuthorShadowRoot() const { return m_hasAuthorShadowRoot; }
-
     double min() const;
     void setMin(double, ExceptionCode&);
 
@@ -68,7 +66,6 @@ private:
     HTMLMeterElement(const QualifiedName&, Document*);
     virtual ~HTMLMeterElement();
 
-    virtual void willAddAuthorShadowRoot() OVERRIDE;
     virtual bool areAuthorShadowsAllowed() const OVERRIDE { return false; }
     RenderMeter* renderMeter() const;
 
@@ -85,7 +82,6 @@ private:
     void createShadowSubtree();
 
     RefPtr<MeterValueElement> m_value;
-    bool m_hasAuthorShadowRoot;
 };
 
 inline bool isHTMLMeterElement(Node* node)
index 6ae809f..84b2e89 100644 (file)
@@ -44,7 +44,6 @@ const double HTMLProgressElement::InvalidPosition = -2;
 HTMLProgressElement::HTMLProgressElement(const QualifiedName& tagName, Document* document)
     : LabelableElement(tagName, document)
     , m_value(0)
-    , m_hasAuthorShadowRoot(false)
 {
     ASSERT(hasTagName(progressTag));
 }
@@ -83,11 +82,6 @@ RenderProgress* HTMLProgressElement::renderProgress() const
     return static_cast<RenderProgress*>(renderObject);
 }
 
-void HTMLProgressElement::willAddAuthorShadowRoot()
-{
-    m_hasAuthorShadowRoot = true;
-}
-
 bool HTMLProgressElement::supportsFocus() const
 {
     return HTMLElement::supportsFocus() && !disabled();
index c39e8dd..33c9614 100644 (file)
@@ -36,8 +36,6 @@ public:
 
     static PassRefPtr<HTMLProgressElement> create(const QualifiedName&, Document*);
 
-    bool hasAuthorShadowRoot() const { return m_hasAuthorShadowRoot; }
-
     double value() const;
     void setValue(double, ExceptionCode&);
 
@@ -54,7 +52,6 @@ private:
     HTMLProgressElement(const QualifiedName&, Document*);
     virtual ~HTMLProgressElement();
 
-    virtual void willAddAuthorShadowRoot() OVERRIDE;
     virtual bool areAuthorShadowsAllowed() const OVERRIDE { return false; }
 
     virtual bool supportLabels() const OVERRIDE { return true; }
@@ -73,7 +70,6 @@ private:
     void createShadowSubtree();
 
     ProgressValueElement* m_value;
-    bool m_hasAuthorShadowRoot;
 };
 
 inline bool isHTMLProgressElement(Node* node)