1681699 - Cover testing hitting early breakpoint during a remote iframe load

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement, P3)

Description

[Alexandre Poirot [:ochameau]]

Bug 1573327 introduces ways to set the breakpoint before the page starts loading.
That, even for page loaded in distinct remote processes, like fission iframes, or when navigating to another process.

We do not have any test yet that cover correctly hitting breakpoints of iframes using another origin and so loading in a distinct process (aka fission iframes).

Bug 1681698 would be a blocker in order to have this test to pass on fission.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1681699 - [devtools] Cover early breakpoint in remote frame with a test.

Comment 2

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

Just making sure this isn't forgotten if not finished this year

Comment 3

[Chris Peterson [:cpeterson]]

Tracking dt-fission-m3-mvp bugs for Fission MVP.

Comment 4

[Alexandre Poirot [:ochameau]]

This bugs ends up listing, via its dependencies, all the necessary work to support the earliest breakpoints.
This includes:

• Initializing the ThreadActor as early as the target starts (bug 1681698)
• Listen to breakpoints via the ResourceWatcher API, use a legacy listener in order to support order devices (bug 1662129)
• Then, implement the server side listening via the watcher actor, so that we can emit "paused" as early as the target starts and frontend can receive it correctly (bug 1683139)

It appears necessary to have breakpoints to become resources as with only bug 1681698, the thread actor emits a "paused" event,
but the frontend miss it as the only way to listen for it is on the threadFront, which is only created after the call the ThreadActor.attach. i.e. way after the "paused" event is received by the client.

Comment 5

[Chris Peterson [:cpeterson]]

Moving some dt-fission-m3-mvp bugs from Fission MVP to M7 (blocking Beta experiment).

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/283763c17ed7
[devtools] Cover early breakpoint in remote frame with a test. r=nchevobbe

Comment 7

[Atila Butkovits]

Backed out for causing failure on browser_browser_resources_console_messages.js.

https://bugzilla.mozilla.org/show_bug.cgi?id=1681698#c11

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9872c90b4b90
[devtools] Cover early breakpoint in remote frame with a test. r=nchevobbe

Comment 9

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/9872c90b4b90