[webkit-reviews] review requested: [Bug 51878] Add WebKit1 API to view and delete local storage : [Attachment 83594] [PATCH] recovery from out-of-sync db, StorageTrackerClient

[bugzilla-daemon at webkit.org]

Anton D'Auria <adauria at apple.com> has asked  for review:
Bug 51878: Add WebKit1 API to view and delete local storage
https://bugs.webkit.org/show_bug.cgi?id=51878

Attachment 83594: [PATCH] recovery from out-of-sync db, StorageTrackerClient
https://bugs.webkit.org/attachment.cgi?id=83594&action=review

------- Additional Comments from Anton D'Auria <adauria at apple.com>
- sync up storage tracker db with files on disk on startup
- implemented StorageTrackerClient for mac
- fix for a race condition in which a local storage db isn't cleared in time
before StorageAreaSync writes new items and cancels the deletion

This is a bit tricky to test with layout tests because StorageTracker is async
and doesn't generate DOM events. I'd like to land this patch with a disabled
StorageTracker until I finish tests.