Changeset 247082 in webkit

Timestamp:

Jul 2, 2019 4:58:24 PM (5 years ago)

Author:

youenn@apple.com

Message:

StorageManager does not need to be a WorkQueueMessageReceiver anymore
​https://bugs.webkit.org/show_bug.cgi?id=199421

Reviewed by Chris Dumez.

• NetworkProcess/WebStorage/StorageManager.h:

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• NetworkProcess/WebStorage/StorageManager.h (modified) (2 diffs)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2019-07-02  Youenn Fablet  <youenn@apple.com>
+
+        StorageManager does not need to be a WorkQueueMessageReceiver anymore
+        https://bugs.webkit.org/show_bug.cgi?id=199421
+
+        Reviewed by Chris Dumez.
+
+        * NetworkProcess/WebStorage/StorageManager.h:
+
 2019-07-02  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebKit/NetworkProcess/WebStorage/StorageManager.h

@@ -46 +46 @@
 using GetValuesCallback = CompletionHandler<void(const HashMap<String, String>&)>;
 
-class StorageManager : public IPC::Connection::WorkQueueMessageReceiver {
+class StorageManager : public ThreadSafeRefCounted<StorageManager> {
 public:
     static Ref<StorageManager> create(const String& localStorageDirectory);
@@ -74 +74 @@
     void getLocalStorageOriginDetails(Function<void(Vector<LocalStorageDatabaseTracker::OriginDetails>&&)>&& completionHandler);
 
-    // IPC::Connection::WorkQueueMessageReceiver.
-    void didReceiveMessage(IPC::Connection&, IPC::Decoder&) override;
-    void didReceiveSyncMessage(IPC::Connection&, IPC::Decoder&, std::unique_ptr<IPC::Encoder>& replyEncoder) override;
+    void didReceiveMessage(IPC::Connection&, IPC::Decoder&);
+    void didReceiveSyncMessage(IPC::Connection&, IPC::Decoder&, std::unique_ptr<IPC::Encoder>& replyEncoder);
 
 private: