Bootstrap thread actor as soon as we start registering breakpoints via the Watcher actor
Categories
(DevTools :: Debugger, enhancement, P3)
Tracking
(Fission Milestone:M7, firefox87 fixed)
| Tracking | Status | |
|---|---|---|
| firefox87 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 2 open bugs)
Details
(Whiteboard: dt-fission-m3-mvp)
Attachments
(1 file, 2 obsolete files)
|
(deleted),
text/x-phabricator-request
|
Details |
Bug 1573327 ensures registering the breakpoints before the process and page starts loading, but this isn't enough to make early breakpoint to work.
For now, the Thread actor is still attached late from the frontend. This prevents setting up the Debugger API early enough and be able to break.
The goal here would be to followup on bug 1573327 and make it so that the Thread actor is "attached", or at least register all what is necessary to make the breakpoint to work.
Ultimately, we would need to get rid of the attach sequence from the frontend and automatically bootstrap the thread actor when the target instantiates, or when we register the breakpoints (this is part of target instantiation).
This is a blocker in order to get breakpoint to work correctly during early fission iframe loads.
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 1•4 years ago
|
||
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
||
Comment 2•4 years ago
|
||
Tracking dt-fission-m3-mvp bugs for Fission MVP.
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 3•4 years ago
|
||
Bin onExceptionUnwind only once, remove unused _scripts attribute, create _debuggerSourcesSeen from constructor.
|
|
Assignee | |
Comment 4•4 years ago
|
||
|
|
Assignee | |
Comment 5•4 years ago
|
||
Moving the patches that can be landed immediately to bug 1686480.
The one patch left, would have to wait for bug 1683139 to be fixed before landing. That's because we would miss early "paused" packets.
Whereas, if pauses are resources, they will no longer be missed. The resource watcher will receive them, even if emitted early during process startup. And the panel, even if not opened yet, will receive them from the ResourceWatcher cache.
|
|
||
Comment 6•4 years ago
|
||
Comment on attachment 9196605 [details]
Bug 1681698 - [devtools] Allow all options currently passed to ThreadActor.attach to be passed via ThreadActor.reconfigure.
Revision D101471 was moved to bug 1686480. Setting attachment 9196605 [details] to obsolete.
|
|
||
Comment 7•4 years ago
|
||
Comment on attachment 9196604 [details]
Bug 1681698 - [devtools] A few naive cleanups in ThreadActor.
Revision D101470 was moved to bug 1686480. Setting attachment 9196604 [details] to obsolete.
|
|
||
Comment 8•4 years ago
|
||
Moving some dt-fission-m3-mvp bugs from Fission MVP to M7 (blocking Beta experiment).
|
|
Assignee | |
Comment 9•4 years ago
|
||
|
||
Comment 10•4 years ago
|
||
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cf75ed5213a0 [devtools] Bootstrap the thread actor early, when the target actor instantiates r=jdescottes,nchevobbe,devtools-backward-compat-reviewers
|
||
Comment 11•4 years ago
|
||
Backed out for causing failure on browser_browser_resources_console_messages.js.
Backout link: https://hg.mozilla.org/integration/autoland/rev/9a783fe8ba4064bdc08905dfe0b00d5efdb2bf51
Failure log: https://treeherder.mozilla.org/logviewer?job_id=329634279&repo=autoland&lineNumber=14398
|
|
||
Comment 12•4 years ago
|
||
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0fc4465fe35d [devtools] Bootstrap the thread actor early, when the target actor instantiates r=jdescottes,nchevobbe,devtools-backward-compat-reviewers
|
||
Comment 13•4 years ago
|
||
| bugherder | ||
Description
•