From c4f893416e7e6e44348e7d7911fffd0dcd87cad0 Mon Sep 17 00:00:00 2001 From: "hayato@chromium.org" Date: Wed, 15 Feb 2012 08:21:29 +0000 Subject: [PATCH] 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 --- Source/WebCore/ChangeLog | 14 ++++++++++++++ Source/WebCore/dom/ShadowRoot.cpp | 2 +- Source/WebCore/dom/ShadowRoot.h | 6 ------ 3 files changed, 15 insertions(+), 7 deletions(-) diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog index 21881ab..0a3bdc9 100644 --- a/Source/WebCore/ChangeLog +++ b/Source/WebCore/ChangeLog @@ -1,3 +1,17 @@ +2012-02-15 Hayato Ito + + 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 Cleanup pending transaction queue in Database. diff --git a/Source/WebCore/dom/ShadowRoot.cpp b/Source/WebCore/dom/ShadowRoot.cpp index 0cb8c9d..ef9ff01 100644 --- a/Source/WebCore/dom/ShadowRoot.cpp +++ b/Source/WebCore/dom/ShadowRoot.cpp @@ -110,7 +110,7 @@ PassRefPtr ShadowRoot::create(Element* element, ShadowRootCreationPu } ASSERT(purpose != CreatingUserAgentShadowRoot || !element->hasShadowRoot()); - RefPtr shadowRoot = create(element->document()); + RefPtr 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 --- 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 { friend class WTF::DoublyLinkedListNode; public: - static PassRefPtr create(Document*); static PassRefPtr create(Element*, ExceptionCode&); // FIXME: We will support multiple shadow subtrees, however current implementation does not work well @@ -95,11 +94,6 @@ private: OwnPtr m_selector; }; -inline PassRefPtr ShadowRoot::create(Document* document) -{ - return adoptRef(new ShadowRoot(document)); -} - inline void ShadowRoot::clearNeedsReattachHostChildrenAndShadow() { m_needsRecalculateContent = false; -- 2.7.4