1687954 - Pause on JS exceptions doesn't work properly in Fission

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Pausing JS execution on JS exceptions (both caught and uncaught) should be properly supported in Fission

STR:

1. Load a page with remote iframe that has JS cause an exception during page load
2. Open DevTools and select the Debugger panel
3. Check "Pause on exceptions" (and "Pause on caught exceptions") in Debugger and reload this page
4. JS execution should pause and the Debugger should show the proper file and line.

Here is a test page:
http://janodvarko.cz/tests/fission/case11/index.html

Honza

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

:Honza, if you think that's a regression, could you try to find a regression range using for example mozregression?

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

(since this blocks M7 bug 1687261, this should also be M7)

Comment 3

[Jan Honza Odvarko [:Honza] (always need-info? me)]

I tested this in a new profile and STRs from comment #0 doesn't work for me.

@bomsy: can you please double check?

Honza

Comment 4

[Hubert Boma Manilla (:bomsy)]

I tried it as well .. it doesn't work for me as well.

Comment 5

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1687954 - [devtools] Add thread configuration actor r=ochameau

Comment 6

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1687954 - [devtools] Use thread configuration actor for POE functionality r=ochameau

Depends on D107116

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/87b87e9f565d
[devtools] Add thread configuration actor r=ochameau,devtools-backward-compat-reviewers
https://hg.mozilla.org/integration/autoland/rev/dacc30c9fdb4
[devtools] Use thread configuration actor for POE functionality r=ochameau

Comment 8

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/87b87e9f565d
https://hg.mozilla.org/mozilla-central/rev/dacc30c9fdb4