Bug 145624 – sourceCodeLocation assert in WebInspector.CallFrame.constructor is wrong

• Component: Web Inspector
• Status: RESOLVED WORKSFORME

[Mark Lam]

Currently, the assertion in WebInspector.CallFrame.constructor expects the sourceCodeLocation to be falsy or be an instance of WebInspector.SourceCodeLocation.  Turns out it can also be an instance of WebInspector.LazySourceCodeLocation.  The assertion is in error.

[Radar WebKit Bug Importer]

<rdar://problem/21234075>

[Nikita Vasilyev]

    class LazySourceCodeLocation extends WebInspector.SourceCodeLocation

So, it should be fine.

When does the assertion fail?

[Mark Lam]

(In reply to comment #2)
>     class LazySourceCodeLocation extends WebInspector.SourceCodeLocation
> 
> So, it should be fine.
> 
> When does the assertion fail?

I reproduced it by running LayoutTests/inspector/debugger/break-on-exceptions.html which is being introduced in bug 145525.

[Nikita Vasilyev]

I tried running tests on r185858 and they all passed. Please let me know if I'm doing something wrong, I don't know much about the tests.


⮀ Tools/Scripts/run-webkit-tests LayoutTests/inspector/debugger/
Using port 'mac-elcapitan-wk2'
Test configuration: <elcapitan, x86_64, release>
Placing test results in /Users/nv/Code/Apple/OpenSource/WebKitBuild/Release/layout-test-results
Baseline search path: mac-wk2 -> wk2 -> mac -> generic
Using Release build
Pixel tests disabled
Regular timeout: 30000, slow test timeout: 150000
Command line: /Users/nv/Code/Apple/OpenSource/WebKitBuild/Release/WebKitTestRunner -

Found 28 tests; running 28, skipping 0.
Running 1 WebKitTestRunner.     

All 28 tests ran as expected.