Followup to "Move WebKit2 AsyncTask to WebCore CrossThreadTask."
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Sep 2015 00:16:31 +0000 (00:16 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Sep 2015 00:16:31 +0000 (00:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=149651

Rubberstamped by Alex Christensen.

Source/WebCore:

No new tests (No change in behavior).

* platform/CrossThreadTask.h: Now in WebCore, use the WebCore namespace.

Source/WebKit2:

* DatabaseProcess/DatabaseProcess.h: CrossThreadTask is in the WebCore namespace, not WebKit.
* DatabaseProcess/IndexedDB/UniqueIDBDatabase.h: Ditto.

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

Source/WebCore/ChangeLog
Source/WebCore/platform/CrossThreadTask.h
Source/WebKit2/ChangeLog
Source/WebKit2/DatabaseProcess/DatabaseProcess.h
Source/WebKit2/DatabaseProcess/IndexedDB/UniqueIDBDatabase.h

index 96cfbe3..8df96f3 100644 (file)
@@ -1,5 +1,16 @@
 2015-09-29  Brady Eidson  <beidson@apple.com>
 
+        Followup to "Move WebKit2 AsyncTask to WebCore CrossThreadTask."
+        https://bugs.webkit.org/show_bug.cgi?id=149651
+
+        Rubberstamped by Alex Christensen.
+
+        No new tests (No change in behavior).
+
+        * platform/CrossThreadTask.h: Now in WebCore, use the WebCore namespace.
+
+2015-09-29  Brady Eidson  <beidson@apple.com>
+
         Move WebKit2 AsyncTask to WebCore CrossThreadTask.
         https://bugs.webkit.org/show_bug.cgi?id=149651
 
index ed40f56..f8dec44 100644 (file)
@@ -29,7 +29,7 @@
 #include "CrossThreadCopier.h"
 #include <functional>
 
-namespace WebKit {
+namespace WebCore {
 
 class CrossThreadTask {
     WTF_MAKE_NONCOPYABLE(CrossThreadTask);
@@ -223,6 +223,6 @@ std::unique_ptr<CrossThreadTask> createCrossThreadTask(
         WebCore::CrossThreadCopier<P8>::copy(parameter8));
 }
 
-} // namespace WebKit
+} // namespace WebCore
 
 #endif // CrossThreadTask_h
index b03833f..1c00ebe 100644 (file)
@@ -1,5 +1,15 @@
 2015-09-29  Brady Eidson  <beidson@apple.com>
 
+        Followup to "Move WebKit2 AsyncTask to WebCore CrossThreadTask."
+        https://bugs.webkit.org/show_bug.cgi?id=149651
+
+        Rubberstamped by Alex Christensen.
+
+        * DatabaseProcess/DatabaseProcess.h: CrossThreadTask is in the WebCore namespace, not WebKit.
+        * DatabaseProcess/IndexedDB/UniqueIDBDatabase.h: Ditto.
+
+2015-09-29  Brady Eidson  <beidson@apple.com>
+
         Move WebKit2 AsyncTask to WebCore CrossThreadTask.
         https://bugs.webkit.org/show_bug.cgi?id=149651
 
index ec47794..c2ff0e6 100644 (file)
 #include <wtf/NeverDestroyed.h>
 
 namespace WebCore {
+class CrossThreadTask;
 class SessionID;
 struct SecurityOriginData;
 }
 
 namespace WebKit {
 
-class CrossThreadTask;
 class DatabaseToWebProcessConnection;
 class UniqueIDBDatabase;
 
@@ -64,7 +64,7 @@ public:
 
     WorkQueue& queue() { return m_queue.get(); }
 
-    void postDatabaseTask(std::unique_ptr<CrossThreadTask>);
+    void postDatabaseTask(std::unique_ptr<WebCore::CrossThreadTask>);
 
 private:
     DatabaseProcess();
@@ -112,7 +112,7 @@ private:
     HashMap<UniqueIDBDatabaseIdentifier, RefPtr<UniqueIDBDatabase>> m_idbDatabases;
 #endif
 
-    Deque<std::unique_ptr<CrossThreadTask>> m_databaseTasks;
+    Deque<std::unique_ptr<WebCore::CrossThreadTask>> m_databaseTasks;
     Lock m_databaseTaskMutex;
 };
 
index 441613e..445e2a8 100644 (file)
@@ -45,6 +45,7 @@ class DataReference;
 }
 
 namespace WebCore {
+class CrossThreadTask;
 class SharedBuffer;
 
 struct IDBDatabaseMetadata;
@@ -56,7 +57,6 @@ struct SecurityOriginData;
 
 namespace WebKit {
 
-class CrossThreadTask;
 class DatabaseProcessIDBConnection;
 class UniqueIDBDatabaseBackingStore;
 
@@ -125,7 +125,7 @@ private:
         Normal,
         Shutdown
     };
-    void postDatabaseTask(std::unique_ptr<CrossThreadTask>, DatabaseTaskType = DatabaseTaskType::Normal);
+    void postDatabaseTask(std::unique_ptr<WebCore::CrossThreadTask>, DatabaseTaskType = DatabaseTaskType::Normal);
 
     void shutdown(UniqueIDBDatabaseShutdownType);
 
@@ -138,11 +138,11 @@ private:
     // Returns true if this origin can use the same databases as the given origin.
     bool canShareDatabases(const WebCore::SecurityOriginData&, const WebCore::SecurityOriginData&) const;
 
-    void postTransactionOperation(const IDBIdentifier& transactionIdentifier, std::unique_ptr<CrossThreadTask>, std::function<void (bool)> successCallback);
+    void postTransactionOperation(const IDBIdentifier& transactionIdentifier, std::unique_ptr<WebCore::CrossThreadTask>, std::function<void (bool)> successCallback);
     
     // To be called from the database workqueue thread only
     void performNextDatabaseTask();
-    void postMainThreadTask(std::unique_ptr<CrossThreadTask>, DatabaseTaskType = DatabaseTaskType::Normal);
+    void postMainThreadTask(std::unique_ptr<WebCore::CrossThreadTask>, DatabaseTaskType = DatabaseTaskType::Normal);
     void openBackingStoreAndReadMetadata(const UniqueIDBDatabaseIdentifier&, const String& databaseDirectory);
     void openBackingStoreTransaction(const IDBIdentifier& transactionIdentifier, const Vector<int64_t>& objectStoreIDs, WebCore::IndexedDB::TransactionMode);
     void beginBackingStoreTransaction(const IDBIdentifier&);
@@ -208,10 +208,10 @@ private:
 
     RefPtr<UniqueIDBDatabaseBackingStore> m_backingStore;
 
-    Deque<std::unique_ptr<CrossThreadTask>> m_databaseTasks;
+    Deque<std::unique_ptr<WebCore::CrossThreadTask>> m_databaseTasks;
     Lock m_databaseTaskMutex;
 
-    Deque<std::unique_ptr<CrossThreadTask>> m_mainThreadTasks;
+    Deque<std::unique_ptr<WebCore::CrossThreadTask>> m_mainThreadTasks;
     Lock m_mainThreadTaskMutex;
 };