Clean up #include <OwnPtr.h>|<PassOwnPtr.h> in Supplementable classes
authorgyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 2 May 2014 08:59:29 +0000 (08:59 +0000)
committergyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 2 May 2014 08:59:29 +0000 (08:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132466

Reviewed by Tim Horton.

Since r168144, Supplementable classes don't need to include OwnPtr.h or PassOwnPtr.h.
Clean up those inclusions.

No new tests, just clean up patch.

* Modules/encryptedmedia/CDMPrivate.h:
* Modules/gamepad/NavigatorGamepad.cpp:
* Modules/geolocation/GeolocationController.cpp:
(WebCore::GeolocationController::create): Deleted. Don't need to have a factory function.
* Modules/geolocation/GeolocationController.h:
* Modules/mediasource/MediaSource.h:
* Modules/mediastream/UserMediaController.h:
* Modules/notifications/NotificationCenter.h:
* Modules/notifications/NotificationController.cpp:
* Modules/quota/StorageErrorCallback.h:
* Modules/vibration/Vibration.h:
* Modules/webdatabase/Database.cpp:
* Modules/websockets/ThreadableWebSocketChannelClientWrapper.h:
* Modules/websockets/WebSocket.cpp:
* Modules/websockets/WebSocket.h:

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

15 files changed:
Source/WebCore/ChangeLog
Source/WebCore/Modules/encryptedmedia/CDMPrivate.h
Source/WebCore/Modules/gamepad/NavigatorGamepad.cpp
Source/WebCore/Modules/geolocation/GeolocationController.cpp
Source/WebCore/Modules/geolocation/GeolocationController.h
Source/WebCore/Modules/mediasource/MediaSource.h
Source/WebCore/Modules/mediastream/UserMediaController.h
Source/WebCore/Modules/notifications/NotificationCenter.h
Source/WebCore/Modules/notifications/NotificationController.cpp
Source/WebCore/Modules/quota/StorageErrorCallback.h
Source/WebCore/Modules/vibration/Vibration.h
Source/WebCore/Modules/webdatabase/Database.cpp
Source/WebCore/Modules/websockets/ThreadableWebSocketChannelClientWrapper.h
Source/WebCore/Modules/websockets/WebSocket.cpp
Source/WebCore/Modules/websockets/WebSocket.h

index d7f6fbb..74cf0a9 100644 (file)
@@ -1,3 +1,31 @@
+2014-05-02  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        Clean up #include <OwnPtr.h>|<PassOwnPtr.h> in Supplementable classes
+        https://bugs.webkit.org/show_bug.cgi?id=132466
+
+        Reviewed by Tim Horton.
+
+        Since r168144, Supplementable classes don't need to include OwnPtr.h or PassOwnPtr.h.
+        Clean up those inclusions.
+
+        No new tests, just clean up patch.
+
+        * Modules/encryptedmedia/CDMPrivate.h:
+        * Modules/gamepad/NavigatorGamepad.cpp:
+        * Modules/geolocation/GeolocationController.cpp:
+        (WebCore::GeolocationController::create): Deleted. Don't need to have a factory function.
+        * Modules/geolocation/GeolocationController.h:
+        * Modules/mediasource/MediaSource.h:
+        * Modules/mediastream/UserMediaController.h:
+        * Modules/notifications/NotificationCenter.h:
+        * Modules/notifications/NotificationController.cpp:
+        * Modules/quota/StorageErrorCallback.h:
+        * Modules/vibration/Vibration.h:
+        * Modules/webdatabase/Database.cpp:
+        * Modules/websockets/ThreadableWebSocketChannelClientWrapper.h:
+        * Modules/websockets/WebSocket.cpp:
+        * Modules/websockets/WebSocket.h:
+
 2014-05-01  Ryuan Choi  <ryuan.choi@samsung.com>
 
         Remove TiledBackingStore stuff from Frame
index 8bb801f..46bd78d 100644 (file)
@@ -28,7 +28,6 @@
 
 #if ENABLE(ENCRYPTED_MEDIA_V2)
 
-#include <wtf/OwnPtr.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
index 4a5e30c..3739467 100644 (file)
@@ -31,7 +31,6 @@
 #include "GamepadList.h"
 #include "Gamepads.h"
 #include "Navigator.h"
-#include <wtf/PassOwnPtr.h>
 
 namespace WebCore {
 
index 974ce94..a11dcec 100644 (file)
@@ -47,11 +47,6 @@ GeolocationController::~GeolocationController()
         m_client->geolocationDestroyed();
 }
 
-PassOwnPtr<GeolocationController> GeolocationController::create(GeolocationClient* client)
-{
-    return adoptPtr(new GeolocationController(client));
-}
-
 void GeolocationController::addObserver(Geolocation* observer, bool enableHighAccuracy)
 {
     // This may be called multiple times with the same observer, though removeObserver()
index 58be3b4..36e1de9 100644 (file)
@@ -47,8 +47,6 @@ public:
     explicit GeolocationController(GeolocationClient*);
     ~GeolocationController();
 
-    static PassOwnPtr<GeolocationController> create(GeolocationClient*);
-
     void addObserver(Geolocation*, bool enableHighAccuracy);
     void removeObserver(Geolocation*);
 
index dc591c0..1d232f4 100644 (file)
@@ -42,7 +42,6 @@
 #include "SourceBuffer.h"
 #include "SourceBufferList.h"
 #include "URLRegistry.h"
-#include <wtf/PassOwnPtr.h>
 #include <wtf/RefCounted.h>
 #include <wtf/Vector.h>
 
index d184b32..703ff40 100644 (file)
@@ -30,7 +30,6 @@
 #include "Page.h"
 #include "UserMediaClient.h"
 #include "UserMediaRequest.h"
-#include <wtf/PassOwnPtr.h>
 
 namespace WebCore {
 
index 09d9b06..70acc4f 100644 (file)
@@ -37,7 +37,6 @@
 #include "ScriptExecutionContext.h"
 #include "Timer.h"
 #include "VoidCallback.h"
-#include <wtf/OwnPtr.h>
 #include <wtf/PassRefPtr.h>
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>
index fe5a0ad..1f1bc82 100644 (file)
@@ -29,7 +29,6 @@
 #if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
 
 #include "NotificationClient.h"
-#include <wtf/PassOwnPtr.h>
 
 namespace WebCore {
 
index 56f3bf6..d3b74ba 100644 (file)
@@ -34,7 +34,6 @@
 #if ENABLE(QUOTA)
 
 #include "ScriptExecutionContext.h"
-#include <wtf/PassOwnPtr.h>
 #include <wtf/RefCounted.h>
 
 namespace WebCore {
index d5f1fa9..cc529a0 100644 (file)
@@ -24,7 +24,6 @@
 
 #include "Page.h"
 #include "Timer.h"
-#include <wtf/PassOwnPtr.h>
 
 namespace WebCore {
 
index ef1e333..bda39cb 100644 (file)
@@ -53,7 +53,6 @@
 #include "ScriptExecutionContext.h"
 #include "SecurityOrigin.h"
 #include "VoidCallback.h"
-#include <wtf/PassOwnPtr.h>
 #include <wtf/PassRefPtr.h>
 #include <wtf/RefPtr.h>
 #include <wtf/StdLibExtras.h>
index 976c99e..b1e2fcc 100644 (file)
@@ -39,7 +39,6 @@
 #include "WorkerThreadableWebSocketChannel.h"
 #include <memory>
 #include <wtf/Forward.h>
-#include <wtf/OwnPtr.h>
 #include <wtf/PassOwnPtr.h>
 #include <wtf/Threading.h>
 #include <wtf/Vector.h>
index cbb529d..837d547 100644 (file)
@@ -57,7 +57,6 @@
 #include <runtime/ArrayBuffer.h>
 #include <runtime/ArrayBufferView.h>
 #include <wtf/HashSet.h>
-#include <wtf/OwnPtr.h>
 #include <wtf/PassOwnPtr.h>
 #include <wtf/StdLibExtras.h>
 #include <wtf/text/CString.h>
index 75482d8..0a68901 100644 (file)
@@ -41,7 +41,6 @@
 #include "WebSocketChannel.h"
 #include "WebSocketChannelClient.h"
 #include <wtf/Forward.h>
-#include <wtf/OwnPtr.h>
 #include <wtf/RefCounted.h>
 #include <wtf/text/AtomicStringHash.h>