710219 - Separate debugger JS modules from plain scripts

Status: RESOLVED FIXED

(DevTools :: Debugger, defect)

Description

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

The debugger JS files that are not modules should not be installed with the others and should be loaded with simple chrome URLs. The list includes: dbg-server.js, dbg-transport.js, dbg-browser-actors.js and dbg-script-actors.js.

Comment 1

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

Attachment: WIP

Moved the scripts and modules under a devtools/ prefix as well. This is not working yet for some reason, but I'll figure it out eventually.

Comment 2

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

Comment on attachment 581305 [details] [diff] [review]
WIP

Weird, it worked fine today after clobbering the build once more. Had I known that yesterday, I wouldn't have split this out into a separate bug.

I'm asking both of you for review in the spirit of landing the debugger with at least two peer reviews.

Comment 3

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 581305 [details] [diff] [review]
WIP

that is a whole lotta load (insert led zeppelin guitar track)

Comment 4

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

https://hg.mozilla.org/users/dcamp_campd.org/remote-debug/rev/6fdee4681f7e