RESOLVED FIXED 128018
Expose creation and modification times for LocalStorage 
https://bugs.webkit.org/show_bug.cgi?id=128018
Summary Expose creation and modification times for LocalStorage
Alexey Proskuryakov
Reported 2014-01-31 14:26:41 PST
<rdar://problem/15642344>
Attachments
proposed patch (21.42 KB, patch)
2014-01-31 14:30 PST, Alexey Proskuryakov 		no flags
DetailsFormatted DiffDiff
with new file added (23.53 KB, patch)
2014-01-31 14:39 PST, Alexey Proskuryakov 		andersca: review+
DetailsFormatted DiffDiff
patch for landing (23.62 KB, patch)
2014-01-31 14:49 PST, Alexey Proskuryakov 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2014-01-31 14:30:40 PST
Created attachment 222851 [details] proposed patch
Alexey Proskuryakov
Comment 2 2014-01-31 14:39:23 PST
Created attachment 222854 [details] with new file added
Anders Carlsson
Comment 3 2014-01-31 14:46:45 PST
Comment on attachment 222854 [details] with new file added View in context: https://bugs.webkit.org/attachment.cgi?id=222854&action=review > Source/WebKit2/UIProcess/Storage/StorageManager.cpp:429 > +void StorageManager::getStoragesByOrigin(FunctionDispatcher* callbackDispatcher, void* context, void (*callback)(const Vector<LocalStorageDetails>& storageDetails, void* context)) I don't think "storages" is a word. How about "storageDetails" or "storageInformation" > Source/WebKit2/UIProcess/Storage/StorageManager.cpp:644 > +void StorageManager::getStoragesByOriginInternal(FunctionDispatcher* dispatcher, void* context, void (*callbackFunction)(const Vector<LocalStorageDetails>& storageDetails, void* context)) Same comment here.
Alexey Proskuryakov
Comment 4 2014-01-31 14:49:39 PST
Created attachment 222856 [details] patch for landing
WebKit Commit Bot
Comment 5 2014-01-31 15:24:02 PST
Comment on attachment 222856 [details] patch for landing Clearing flags on attachment: 222856 Committed r163212: <http://trac.webkit.org/changeset/163212>
WebKit Commit Bot
Comment 6 2014-01-31 15:24:04 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.