Summary: | Remove session storage setting | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeremy Orlow <jorlow> | ||||
Component: | New Bugs | Assignee: | Jeremy Orlow <jorlow> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Jeremy Orlow
2009-11-09 16:45:27 PST
Created attachment 42812 [details]
Patch
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?)
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. Committed r50754: <http://trac.webkit.org/changeset/50754> |