[webkit-reviews] review denied: [Bug 37717] Provide a way for clients to set session storage quota : [Attachment 53583] Revised patch with additional comments in Settings.h and ChangeLog.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Apr 16 17:14:12 PDT 2010


Jeremy Orlow <jorlow at chromium.org> has denied Ada Chan <adachan at apple.com>'s
request for review:
Bug 37717: Provide a way for clients to set session storage quota
https://bugs.webkit.org/show_bug.cgi?id=37717

Attachment 53583: Revised patch with additional comments in Settings.h and
ChangeLog.
https://bugs.webkit.org/attachment.cgi?id=53583&action=review

------- Additional Comments from Jeremy Orlow <jorlow at chromium.org>
I don't think your comments adequately explain the situation.  This helps one
element of memory consumption.	And the only reason why it's interesting is
that the memory used stays until the Page dies.  If that weren't true, I'd r-
and mark the bug invalid.

LocalStorage and SessionStorage are often grouped together in peoples mind.  So
I want to make sure that if someone's using LocalStorage quota they don't
automatically think using SessionStorage quota is important to them.


More information about the webkit-reviews mailing list