Title: [143332] trunk/Source/WebKit2
Revision
143332
Author
ch.du...@sisa.samsung.com
Date
2013-02-19 07:53:51 -0800 (Tue, 19 Feb 2013)

Log 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):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (143331 => 143332)


--- trunk/Source/WebKit2/ChangeLog	2013-02-19 15:36:16 UTC (rev 143331)
+++ trunk/Source/WebKit2/ChangeLog	2013-02-19 15:53:51 UTC (rev 143332)
@@ -1,3 +1,18 @@
+2013-02-19  Christophe Dumez  <ch.du...@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  <cgar...@igalia.com>
 
         [GTK] Remove webkit_web_view_get_subresources from WebKit2 GTK+ API

Modified: trunk/Source/WebKit2/UIProcess/Storage/StorageManager.cpp (143331 => 143332)


--- trunk/Source/WebKit2/UIProcess/Storage/StorageManager.cpp	2013-02-19 15:36:16 UTC (rev 143331)
+++ trunk/Source/WebKit2/UIProcess/Storage/StorageManager.cpp	2013-02-19 15:53:51 UTC (rev 143332)
@@ -113,12 +113,12 @@
 
 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)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to