Regression(r143273): Caused a lot of tests to time out
authorch.dumez@sisa.samsung.com <ch.dumez@sisa.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Feb 2013 15:53:51 +0000 (15:53 +0000)
committerch.dumez@sisa.samsung.com <ch.dumez@sisa.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Feb 2013 15:53:51 +0000 (15:53 +0000)
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):

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

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

index 6d1bbf1..6d94ffa 100644 (file)
@@ -1,3 +1,18 @@
+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>
 
         [GTK] Remove webkit_web_view_get_subresources from WebKit2 GTK+ API
index 8c4df1a..42e13b9 100644 (file)
@@ -113,12 +113,12 @@ void StorageManager::createSessionStorageNamespace(uint64_t storageNamespaceID)
 
 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));
 }
 
 void StorageManager::processWillOpenConnection(WebProcessProxy* webProcessProxy)