1117940 - debugger statement in the eval'ed scripts does not work on page reload

Status: RESOLVED FIXED

(DevTools :: Debugger, defect)

Description

[Yury Delendik (:yury)]

Attachment: debug.html

STR:
1. Open Firefox (new process and new profile) and its devtools on Debugger tab;
2. Open the attached debug.html
3. Wait when script breaks
4. Hit Cmd+R to reload page multiple times

Notice that debugger does not break anymore and more than one "debug.html > eval" appeared in the list of the scripts.

Comment 1

[Yury Delendik (:yury)]

Noticed error message in the console:

Handler function DebuggerProgressListener.prototype.onStateChange threw an exception: [Exception... "Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIRequest.suspend]"  nsresult: "0x80040111 (NS_ERROR_NOT_AVAILABLE)"  location: "JS frame :: resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js :: TabActor.prototype._willNavigate :: line 1430"  data: no]
Stack: TabActor.prototype._willNavigate@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js:1430:6
DebuggerProgressListener.prototype.onStateChange<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js:2139:6
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:13
TabActor.prototype.onReload/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/webbrowser.js:1151:0
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:13
EventLoop.prototype.enter@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:363:4
ThreadActor.prototype._pushThreadPause@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:572:4
ThreadActor.prototype._pauseAndRespond@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:771:6
ThreadActor.prototype.onDebuggerStatement@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:1898:8
anonymous/<@https://bug1117940.bugzilla.mozilla.org/attachment.cgi?id=8544155 line 8 > Function:1:0
@https://bug1117940.bugzilla.mozilla.org/attachment.cgi?id=8544155:9:4
Line: 1430, column: 0

Comment 2

[Yury Delendik (:yury)]

behavior in e10s is as defined above. With non-e10s I see only two "> eval" script, that look the same. Which one will allow set breakpoints?

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

CCing James for comment 2.

Comment 4

[James Long (:jlongster)]

I can't reproduce this, and I don't really know how to parse XPCOM errors (isn't that from XPCOM? certainly from the platform component layer).

(In reply to Yury Delendik (:yury) from comment #2)
> behavior in e10s is as defined above. With non-e10s I see only two "> eval"
> script, that look the same. Which one will allow set breakpoints?

I also don't see his, I see the original HTML file and the eval script: http://jlongster.com/s/function-eval.png

Any particular build settings or context I should try?

Comment 5

[Yury Delendik (:yury)]

Attachment: Screencast to reproduce the bug (non-e10s)

Comment 6

[Yury Delendik (:yury)]

Attachment: Screencast to reproduce the bug (part 2, e10s)

Comment 7

[Jason Laster [:jlast]]

Yury, does this still happen for you?

Comment 8

[Jason Laster [:jlast]]

this works for me now