1380510 - Error thrown in watch expression prevents loading proper source

Status: RESOLVED FIXED

(DevTools :: Debugger, defect)

Description

[Oriol Brufau [:Oriol]]

1. Open the new debugger frontend
2. Add a watch expression which throws an error, e.g.
    null.foo
3. Go to the console tab and enter
    debugger; 123;
4. The debugger shows a SOURCEsource1272 file which contains null.foo instead of debugger; 123;

Comment 1

[Jason Laster [:jlast]]

Thanks for sharing. This is a legitimate bug. I knew we did something similar when the debugger setting "pause on exceptions" is enabled, but I was not aware of this case. 

We should be able to fix this soon.

Comment 2

[Jason Laster [:jlast]]

I filed an issue in github here: https://github.com/devtools-html/debugger.html/issues/3329

Comment 3

[Jason Laster [:jlast]]

Oriol, we made some fixes to watch expressions, which landed last night.

https://shipusercontent.com/2fec783060bff6b80c951194dae9962e/Screen%20Shot%202017-09-29%20at%2010.26.23%20AM.png

I think this bug is fixed. There could very well be other watch expressions bugs with identical expressions

Comment 4

[Oriol Brufau [:Oriol]]

It seems this was already fixed by bug 1383803. I think the recent changes in watch expressions just create more inconsistencies without fixing the underlying problems.