1608078 - Stepping out of async function doesn't work properly

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P2)

Description

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

STR:

1. Open DevTools Toolbox & Debugger
2. Load: http://janodvarko.cz/tests/debugger/async-callstacks/
3. Click on button from Test case #1 -> you should pause the JS execution
4. Click Step out -> the JS execution is resumed => BUG

Expected result:
The debugger should pause on line 7 => console.log(data);, which is the line next the the loadData function call.

Honza

Comment 1

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

Jason, you mentioned that you want to work on this.
Here is a nice & simple test case.

Honza

Comment 2

[Jason Laster [:jlast]]

Attachment: Bug 1608078 - Stepping out of async function doesnt work properly. r=loganfsmyth

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7050d098119c
Stepping out of async function doesnt work properly. r=loganfsmyth

Comment 4

[Natalia Csoregi [:nataliaCs]]

Backed out for failures on test_stepping-12.js

backout: https://hg.mozilla.org/integration/autoland/rev/c3ac91c63eeb6037e6c07591d198460703e92279

push: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=7050d098119c2dc261011291ffdf7a90318de3f0&searchStr=xpcshell&selectedTaskRun=HNXWGQ1SQUeTpCh3L425IA-0

failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=303150981&repo=autoland&lineNumber=3058

[task 2020-05-20T20:35:47.175Z] 20:35:47 INFO - TEST-START | devtools/server/tests/xpcshell/test_stepping-12.js
[task 2020-05-20T20:40:47.179Z] 20:40:47 WARNING - TEST-UNEXPECTED-TIMEOUT | devtools/server/tests/xpcshell/test_stepping-12.js | Test timed out
[task 2020-05-20T20:40:47.180Z] 20:40:47 INFO - TEST-INFO took 300000ms
[task 2020-05-20T20:40:47.184Z] 20:40:47 INFO - >>>>>>>
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - (xpcshell/head.js) | test MAIN run_test pending (1)
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - (xpcshell/head.js) | test run_next_test 0 pending (2)
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - (xpcshell/head.js) | test MAIN run_test finished (2)
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - running event loop
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - devtools/server/tests/xpcshell/test_stepping-12.js | Starting
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - (xpcshell/head.js) | test pending (2)
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - PID 19016 | >>> Run thread front test against a regular DevToolsServer
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - (xpcshell/head.js) | test run_next_test 0 finished (2)
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - TEST-PASS | devtools/server/tests/xpcshell/test_stepping-12.js | - Thread client is paused - "paused" == "paused"
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - TEST-PASS | devtools/server/tests/xpcshell/test_stepping-12.js | - true == true
[task 2020-05-20T20:40:47.185Z] 20:40:47 INFO - TEST-PASS | devtools/server/tests/xpcshell/test_stepping-12.js | - "attached" == "attached"

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/983b9b9624c0
Stepping out of async function doesnt work properly. r=loganfsmyth

Comment 6

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/983b9b9624c0