Bug 110163

Summary: Add StorageManager member functions for keeping track of session storage namespaces
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 110212    
Bug Blocks:    
Attachments:
Description Flags
Patch kling: review+

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.