Bug 31279 - Remove session storage setting
Summary: Remove session storage setting
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Jeremy Orlow
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-09 16:45 PST by Jeremy Orlow
Modified: 2009-11-10 12:53 PST (History)
0 users

See Also:


Attachments
Patch (3.42 KB, patch)
2009-11-09 16:48 PST, Jeremy Orlow
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Orlow 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
Comment 1 Jeremy Orlow 2009-11-09 16:48:48 PST
Created attachment 42812 [details]
Patch
Comment 2 Eric Seidel (no email) 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?)
Comment 3 Jeremy Orlow 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.
Comment 4 Jeremy Orlow 2009-11-10 12:53:36 PST
Committed r50754: <http://trac.webkit.org/changeset/50754>