ShadowRoot: Remove a public static factory function which doesn't have any callers.

https://bugs.webkit.org/show_bug.cgi?id=78668

Reviewed by Kent Tamura.

No tests. No change in behavior.

* dom/ShadowRoot.cpp:
(WebCore::ShadowRoot::create):
* dom/ShadowRoot.h:
(ShadowRoot):

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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 21881ab..0a3bdc9 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2012-02-15  Hayato Ito  <hayato@chromium.org>
+
+        ShadowRoot: Remove a public static factory function which doesn't have any callers.
+        https://bugs.webkit.org/show_bug.cgi?id=78668
+
+        Reviewed by Kent Tamura.
+
+        No tests. No change in behavior.
+
+        * dom/ShadowRoot.cpp:
+        (WebCore::ShadowRoot::create):
+        * dom/ShadowRoot.h:
+        (ShadowRoot):
+
 2012-02-14  Hao Zheng  <zhenghao@chromium.org>
 
         Cleanup pending transaction queue in Database.

diff --git a/Source/WebCore/dom/ShadowRoot.cpp b/Source/WebCore/dom/ShadowRoot.cpp
index 0cb8c9d..ef9ff01 100644 (file)
--- a/Source/WebCore/dom/ShadowRoot.cpp
+++ b/Source/WebCore/dom/ShadowRoot.cpp
@@ -110,7 +110,7 @@ PassRefPtr<ShadowRoot> ShadowRoot::create(Element* element, ShadowRootCreationPu
     }
 
     ASSERT(purpose != CreatingUserAgentShadowRoot || !element->hasShadowRoot());
-    RefPtr<ShadowRoot> shadowRoot = create(element->document());
+    RefPtr<ShadowRoot> shadowRoot = adoptRef(new ShadowRoot(element->document()));
 
     ec = 0;
     element->setShadowRoot(shadowRoot, ec);

diff --git a/Source/WebCore/dom/ShadowRoot.h b/Source/WebCore/dom/ShadowRoot.h
index ba453ac..83e1713 100644 (file)
--- a/Source/WebCore/dom/ShadowRoot.h
+++ b/Source/WebCore/dom/ShadowRoot.h
@@ -41,7 +41,6 @@ class HTMLContentSelector;
 class ShadowRoot : public DocumentFragment, public TreeScope, public DoublyLinkedListNode<ShadowRoot> {
     friend class WTF::DoublyLinkedListNode<ShadowRoot>;
 public:
-    static PassRefPtr<ShadowRoot> create(Document*);
     static PassRefPtr<ShadowRoot> create(Element*, ExceptionCode&);
 
     // FIXME: We will support multiple shadow subtrees, however current implementation does not work well
@@ -95,11 +94,6 @@ private:
     OwnPtr<HTMLContentSelector> m_selector;
 };
 
-inline PassRefPtr<ShadowRoot> ShadowRoot::create(Document* document)
-{
-    return adoptRef(new ShadowRoot(document));
-}
-
 inline void ShadowRoot::clearNeedsReattachHostChildrenAndShadow()
 {
     m_needsRecalculateContent = false;