[Webkit-unassigned] [Bug 65582] Web Inspector: Remove InspectorFrontendHost load/saveSessionSetting

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=65582





--- Comment #1 from Pavel Feldman <pfeldman at chromium.org>  2011-08-03 09:13:00 PST ---
(In reply to comment #0)
> It looks like load/saveSessionSetting is only used in one place, for "debugger-enabled".
> In fact, that has been superseded by the WebInspector.setting localStorage setting. It
> looks we can just remove these functions:
> 
>   - JS: InspectorFrontendHostStub, ScriptsPanel.js usage
>   - Inspector: InspectorFrontendHost + Client
>   - WebKit 1 Ports: cf, mac, win and hashmap instance variable

You are probably right: we now have persistent localStorage settings and runtime cookie. This code should vanish. Do you have a patch pending?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.