RESOLVED FIXED 110163
Add StorageManager member functions for keeping track of session storage namespaces 
https://bugs.webkit.org/show_bug.cgi?id=110163
Summary Add StorageManager member functions for keeping track of session storage name...
Anders Carlsson
Reported 2013-02-18 15:53:55 PST
Add StorageManager member functions for keeping track of session storage namespaces
Attachments
Patch (7.09 KB, patch)
2013-02-18 15:55 PST, Anders Carlsson 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2013-02-18 15:55:10 PST
Created attachment 188957 [details] Patch
Anders Carlsson
Comment 2 2013-02-18 16:05:54 PST
Committed r143273: <http://trac.webkit.org/changeset/143273>
Chris Dumez
Comment 3 2013-02-19 05:32:59 PST
Comment on attachment 188957 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188957&action=review > Source/WebKit2/UIProcess/Storage/StorageManager.cpp:56 > + m_queue->dispatch(bind(&StorageManager::destroySessionStorageNamespace, this, storageNamespaceID)); Shouldn't this be destroySessionStorageNamespaceInternal? > Source/WebKit2/UIProcess/Storage/StorageManager.cpp:61 > + m_queue->dispatch(bind(&StorageManager::cloneSessionStorageNamespace, this, storageNamespaceID, newStorageNamespaceID)); Shouldn't this be cloneSessionStorageNamespaceInternal?
Chris Dumez
Comment 4 2013-02-19 05:44:04 PST
I uploaded a patch at Bug 110212 to address this issue as it seems to cause a lot of tests to time out on our bots. StorageManager::cloneSessionStorageNamespace() is called when a new window is opened (for example in Web Inspector tests) and StorageManager::cloneSessionStorageNamespace() just keeps calling itself.
Note You need to log in before you can comment on or make changes to this bug.