Use std::unique_ptr instead of PassOwnPtr|OwnPtr for NamedNodeMap
authorgyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 29 Sep 2014 07:15:14 +0000 (07:15 +0000)
committergyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 29 Sep 2014 07:15:14 +0000 (07:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=137201

Reviewed by Darin Adler.

No new tests, just refactoring patch.

* dom/ElementRareData.h: Use std::unique_ptr instead of OwnPtr.
(WebCore::ElementRareData::setAttributeMap): Use std::unique_ptr.
* dom/NamedNodeMap.h:
(WebCore::NamedNodeMap::create): ditto.

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

Source/WebCore/ChangeLog
Source/WebCore/dom/ElementRareData.h
Source/WebCore/dom/NamedNodeMap.h

index a12b740..013b26a 100644 (file)
@@ -1,3 +1,17 @@
+2014-09-29  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        Use std::unique_ptr instead of PassOwnPtr|OwnPtr for NamedNodeMap
+        https://bugs.webkit.org/show_bug.cgi?id=137201
+
+        Reviewed by Darin Adler.
+
+        No new tests, just refactoring patch.
+
+        * dom/ElementRareData.h: Use std::unique_ptr instead of OwnPtr.
+        (WebCore::ElementRareData::setAttributeMap): Use std::unique_ptr. 
+        * dom/NamedNodeMap.h:
+        (WebCore::NamedNodeMap::create): ditto.
+
 2014-09-28  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Replace wkGetGlyphsForCharacters() with CGFontGetGlyphsForUnichars()
index b575797..735fc2b 100644 (file)
@@ -86,7 +86,7 @@ public:
     void setShadowRoot(PassRefPtr<ShadowRoot> shadowRoot) { m_shadowRoot = shadowRoot; }
 
     NamedNodeMap* attributeMap() const { return m_attributeMap.get(); }
-    void setAttributeMap(PassOwnPtr<NamedNodeMap> attributeMap) { m_attributeMap = attributeMap; }
+    void setAttributeMap(std::unique_ptr<NamedNodeMap> attributeMap) { m_attributeMap = WTF::move(attributeMap); }
 
     RenderStyle* computedStyle() const { return m_computedStyle.get(); }
     void setComputedStyle(PassRef<RenderStyle> computedStyle) { m_computedStyle = WTF::move(computedStyle); }
@@ -140,7 +140,7 @@ private:
     std::unique_ptr<DatasetDOMStringMap> m_dataset;
     std::unique_ptr<ClassList> m_classList;
     RefPtr<ShadowRoot> m_shadowRoot;
-    OwnPtr<NamedNodeMap> m_attributeMap;
+    std::unique_ptr<NamedNodeMap> m_attributeMap;
 
     RefPtr<PseudoElement> m_beforePseudoElement;
     RefPtr<PseudoElement> m_afterPseudoElement;
index 1c48cf8..8273048 100644 (file)
@@ -26,7 +26,6 @@
 #define NamedNodeMap_h
 
 #include "ScriptWrappable.h"
-#include <wtf/PassOwnPtr.h>
 #include <wtf/PassRefPtr.h>
 #include <wtf/text/AtomicString.h>
 
@@ -41,9 +40,9 @@ class NamedNodeMap : public ScriptWrappable {
     WTF_MAKE_FAST_ALLOCATED;
     friend class Element;
 public:
-    static PassOwnPtr<NamedNodeMap> create(Element& element)
+    static std::unique_ptr<NamedNodeMap> create(Element& element)
     {
-        return adoptPtr(new NamedNodeMap(element));
+        return std::unique_ptr<NamedNodeMap>(new NamedNodeMap(element));
     }
 
     void ref();