RESOLVED FIXED94647
Web Inspector: [regression] Settings panel fails to open 
https://bugs.webkit.org/show_bug.cgi?id=94647
Summary Web Inspector: [regression] Settings panel fails to open
johnjbarton
Reported 2012-08-21 16:39:16 PDT
After Bug 94416 - Web Inspector: load scripts panel lazily the Settings fails to open unless the user happens to open the Sources panel first. SettingScreen.js uses WebInspector.settings.searchInContentScripts p.appendChild(this._createCheckboxSetting(WebInspector.UIString("Search in content scripts"), WebInspector.settings.searchInContentScripts)); But the value is not define until ScriptsPanel.js imports ScriptsSearchScope.js Source/WebCore/inspector/front-end/ScriptsPanel.js:importScript("ScriptsSearchScope.js"); which creates the setting: WebInspector.settings.searchInContentScripts = WebInspector.settings.createSetting("searchInContentScripts", false);
Attachments
Patch (2.21 KB, patch)
2012-08-21 19:37 PDT, Pavel Feldman 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Pavel Feldman
Comment 1 2012-08-21 19:20:10 PDT
I'l fix it right away. Now that it squeezed into the Canary, the workaround is to open Sources tab first. Then settings will open.
Pavel Feldman
Comment 2 2012-08-21 19:37:49 PDT
Created attachment 159844 [details] Patch
Pavel Feldman
Comment 3 2012-08-21 19:39:44 PDT
Committed r126253: <http://trac.webkit.org/changeset/126253>
Note You need to log in before you can comment on or make changes to this bug.