This error started showing up after r185213: <http://trac.webkit.org/changeset/185213>. The complete error message says: Uncaught exception in inspector page: TypeError: undefined is not an object (evaluating 'WebInspector.clearLogOnReload.value') [file:///.../OpenSource/WebKitBuild/Debug/WebInspectorUI.framework/Resources/Controllers/LogManager.js:77] I see that WebInspector.clearLogOnReload is initialized by WebInspector.loaded() in Main.js. Are we guaranteed that WebInspector.loaded will be called before the first use of WebInspector.clearLogOnReload? I'm seeing the above error while trying to run new inspector tests I'm writing to test exception breakpoints.
<rdar://problem/21260137>
To reproduce this issue: 1. Comment out the asserts in InspectorTimelineAgent.cpp if the fix for https://bugs.webkit.org/show_bug.cgi?id=145090 is not yet landed. 2. Run the following inspector tests (landed in r185259): OpenSource$ ./Tools/Scripts/run-webkit-tests -1 --debug inspector/debugger/break-on-* That should run a set of 20 tests. The issue will manifest intermittently on a subset of those tests, and result in failures due to text diffs.
(In reply to comment #0) > This error started showing up after r185213: > <http://trac.webkit.org/changeset/185213>. The complete error message says: > > Uncaught exception in inspector page: TypeError: undefined is not an object > (evaluating 'WebInspector.clearLogOnReload.value') > [file:///.../OpenSource/WebKitBuild/Debug/WebInspectorUI.framework/Resources/ > Controllers/LogManager.js:77] It sounds like you reproduced this running tests. I suspect this is a Tests only issue. We just need to add a fake WebInspector.clearLogOnReload WebInspector.Setting to Test.js. Or, move it to LogManager like I suggested. I'll take a quick look.
Created attachment 254389 [details] [PATCH] Proposed Fix
Comment on attachment 254389 [details] [PATCH] Proposed Fix r=me
Comment on attachment 254389 [details] [PATCH] Proposed Fix Clearing flags on attachment: 254389 Committed r185271: <http://trac.webkit.org/changeset/185271>
All reviewed patches have been landed. Closing bug.