Changeset 168252 in webkit

Timestamp:

May 4, 2014 6:16:18 PM (10 years ago)

Author:

gyuyoung.kim@samsung.com

Message:

Use std::unique_ptr in m_taskTimer of Notification class
​https://bugs.webkit.org/show_bug.cgi?id=132544

Reviewed by Andreas Kling.

No new tests, no behavior change.

• Modules/notifications/Notification.cpp:

(WebCore::Notification::Notification):

• Modules/notifications/Notification.h: Use std::unique_ptr instead of OwnPtr.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• Modules/notifications/Notification.cpp (modified) (1 diff)
• Modules/notifications/Notification.h (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-05-04  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        Use std::unique_ptr in m_taskTimer of Notification class
+        https://bugs.webkit.org/show_bug.cgi?id=132544
+
+        Reviewed by Andreas Kling.
+
+        No new tests, no behavior change.
+
+        * Modules/notifications/Notification.cpp:
+        (WebCore::Notification::Notification):
+        * Modules/notifications/Notification.h: Use std::unique_ptr instead of OwnPtr.
+
 2014-05-04  Tim Horton  <timothy_horton@apple.com>
 

trunk/Source/WebCore/Modules/notifications/Notification.cpp

@@ -85 +85 @@
     , m_title(title)
     , m_state(Idle)
-    , m_taskTimer(adoptPtr(new Timer<Notification>(this, &Notification::taskTimerFired)))
+    , m_taskTimer(std::make_unique<Timer<Notification>>(this, &Notification::taskTimerFired))
 {
     m_notificationCenter = DOMWindowNotifications::webkitNotifications(toDocument(context).domWindow());

trunk/Source/WebCore/Modules/notifications/Notification.h

@@ -41 +41 @@
 #include "TextDirection.h"
 #include "ThreadableLoaderClient.h"
-#include <wtf/OwnPtr.h>
 #include <wtf/PassRefPtr.h>
 #include <wtf/RefCounted.h>
@@ -182 +181 @@
 
 #if ENABLE(NOTIFICATIONS)
-    OwnPtr<Timer<Notification>> m_taskTimer;
+    std::unique_ptr<Timer<Notification>> m_taskTimer;
 #endif
 };