Quick follow-up to last check-in, addressing review comments.
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Jan 2015 17:30:16 +0000 (17:30 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Jan 2015 17:30:16 +0000 (17:30 +0000)
* html/parser/HTMLStackItem.h:
(WebCore::HTMLStackItem::HTMLStackItem): Use rvalue reference for incoming Ref.
(WebCore::HTMLStackItem::create): Ditto.

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

Source/WebCore/ChangeLog
Source/WebCore/html/parser/HTMLStackItem.h

index b609d4da3eb0f341202d14e58c862591280503c4..a236e7e2b972ee7718ad247fd3a92f64cff62b50 100644 (file)
@@ -1,3 +1,11 @@
+2015-01-05  Darin Adler  <darin@apple.com>
+
+        Quick follow-up to last check-in, addressing review comments.
+
+        * html/parser/HTMLStackItem.h:
+        (WebCore::HTMLStackItem::HTMLStackItem): Use rvalue reference for incoming Ref.
+        (WebCore::HTMLStackItem::create): Ditto.
+
 2015-01-05  Darin Adler  <darin@apple.com>
 
         Modernize and streamline HTMLStackItem
index dd6a22795e9f6e9158ea407ff18574400c12cc4e..e339771605298144e0669b203ec6b38ac51aa38b 100644 (file)
@@ -39,7 +39,7 @@ namespace WebCore {
 class HTMLStackItem : public RefCounted<HTMLStackItem> {
 public:
     // Normal HTMLElementStack and HTMLFormattingElementList items.
-    static Ref<HTMLStackItem> create(Ref<Element>, AtomicHTMLToken&, const AtomicString& namespaceURI = HTMLNames::xhtmlNamespaceURI);
+    static Ref<HTMLStackItem> create(Ref<Element>&&, AtomicHTMLToken&, const AtomicString& namespaceURI = HTMLNames::xhtmlNamespaceURI);
 
     // Document fragment or element for parsing context.
     static Ref<HTMLStackItem> create(Element&);
@@ -61,7 +61,7 @@ public:
     bool matchesHTMLTag(const AtomicString&) const;
 
 private:
-    HTMLStackItem(Ref<Element>, AtomicHTMLToken&, const AtomicString& namespaceURI);
+    HTMLStackItem(Ref<Element>&&, AtomicHTMLToken&, const AtomicString& namespaceURI);
     explicit HTMLStackItem(Element&);
     explicit HTMLStackItem(DocumentFragment&);
 
@@ -75,7 +75,7 @@ bool isInHTMLNamespace(const HTMLStackItem&);
 bool isNumberedHeaderElement(const HTMLStackItem&);
 bool isSpecialNode(const HTMLStackItem&);
 
-inline HTMLStackItem::HTMLStackItem(Ref<Element> element, AtomicHTMLToken& token, const AtomicString& namespaceURI = HTMLNames::xhtmlNamespaceURI)
+inline HTMLStackItem::HTMLStackItem(Ref<Element>&& element, AtomicHTMLToken& token, const AtomicString& namespaceURI = HTMLNames::xhtmlNamespaceURI)
     : m_node(WTF::move(element))
     , m_namespaceURI(namespaceURI)
     , m_localName(token.name())
@@ -84,7 +84,7 @@ inline HTMLStackItem::HTMLStackItem(Ref<Element> element, AtomicHTMLToken& token
     // FIXME: We should find a way to move the attributes vector in the normal code path instead of copying it.
 }
 
-inline Ref<HTMLStackItem> HTMLStackItem::create(Ref<Element> element, AtomicHTMLToken& token, const AtomicString& namespaceURI)
+inline Ref<HTMLStackItem> HTMLStackItem::create(Ref<Element>&& element, AtomicHTMLToken& token, const AtomicString& namespaceURI)
 {
     return adoptRef(*new HTMLStackItem(WTF::move(element), token, namespaceURI));
 }