RESOLVED FIXED 31279
Remove session storage setting 
https://bugs.webkit.org/show_bug.cgi?id=31279
Summary Remove session storage setting
Jeremy Orlow
Reported 2009-11-09 16:45:27 PST
Remove session storage from Settings. It was added temporarily so we could disable it by default at runtime in Chromium. We now disable these things in a different way, so it's time to remove it. Qt also depended on this setting for a short period of time, but after talking to them we agreed that it should be removed. This bug is the second half of https://bugs.webkit.org/show_bug.cgi?id=30602
Attachments
Patch (3.42 KB, patch)
2009-11-09 16:48 PST, Jeremy Orlow 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jeremy Orlow
Comment 1 2009-11-09 16:48:48 PST
Created attachment 42812 [details] Patch
Eric Seidel (no email)
Comment 2 2009-11-09 17:43:34 PST
Comment on attachment 42812 [details] Patch How is it disabled now? Are none of the WebKit ports affected by this change? (i.e. this call was only used within WebCore itself?)
Jeremy Orlow
Comment 3 2009-11-09 18:06:34 PST
QT and Chromium were the only ones that ever depended on it and they no longer do. There's really no good reason why anyone would want to disable this feature when embedding WebKit and it's always been on by default anyway. The flag was originally added because it was the best way to hide features from web pages while we were still getting them stable within Chromium and it mirrored how we LocalStorage is disabled (even though it's clear that setting is somewhat of an abuse since it can only half be disabled once a page uses LocalStorage and all pages in the page group need to share the same path and such). It's also worth noting that we no longer use this method for adding features behind a runtime flag.
Jeremy Orlow
Comment 4 2009-11-10 12:53:36 PST
Committed r50754: <http://trac.webkit.org/changeset/50754>
Note You need to log in before you can comment on or make changes to this bug.