Changeset 143332 in webkit

Timestamp:

Feb 19, 2013 7:53:51 AM (11 years ago)

Author:

Christophe Dumez

Message:

Regression(r143273): Caused a lot of tests to time out
​https://bugs.webkit.org/show_bug.cgi?id=110212

Reviewed by Anders Carlsson.

StorageManager::destroySessionStorageNamespace() and StorageManager::cloneSessionStorageNamespace()
should call their respective "*Internal" equivalent instead of calling themselves. This lead to
a lot of time outs on our build bots.

• UIProcess/Storage/StorageManager.cpp:

(WebKit::StorageManager::destroySessionStorageNamespace):
(WebKit::StorageManager::cloneSessionStorageNamespace):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/Storage/StorageManager.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-02-19  Christophe Dumez  <ch.dumez@sisa.samsung.com>
+
+        Regression(r143273): Caused a lot of tests to time out
+        https://bugs.webkit.org/show_bug.cgi?id=110212
+
+        Reviewed by Anders Carlsson.
+
+        StorageManager::destroySessionStorageNamespace() and StorageManager::cloneSessionStorageNamespace()
+        should call their respective "*Internal" equivalent instead of calling themselves. This lead to
+        a lot of time outs on our build bots.
+
+        * UIProcess/Storage/StorageManager.cpp:
+        (WebKit::StorageManager::destroySessionStorageNamespace):
+        (WebKit::StorageManager::cloneSessionStorageNamespace):
+
 2013-02-18  Carlos Garcia Campos  <cgarcia@igalia.com>
 

trunk/Source/WebKit2/UIProcess/Storage/StorageManager.cpp

@@ -114 +114 @@
 void StorageManager::destroySessionStorageNamespace(uint64_t storageNamespaceID)
 {
-    m_queue->dispatch(bind(&StorageManager::destroySessionStorageNamespace, this, storageNamespaceID));
+    m_queue->dispatch(bind(&StorageManager::destroySessionStorageNamespaceInternal, this, storageNamespaceID));
 }
 
 void StorageManager::cloneSessionStorageNamespace(uint64_t storageNamespaceID, uint64_t newStorageNamespaceID)
 {
-    m_queue->dispatch(bind(&StorageManager::cloneSessionStorageNamespace, this, storageNamespaceID, newStorageNamespaceID));
+    m_queue->dispatch(bind(&StorageManager::cloneSessionStorageNamespaceInternal, this, storageNamespaceID, newStorageNamespaceID));
 }