1663706 - Early console message made in remote frame are missing

Status: RESOLVED DUPLICATE of bug 1667362

(DevTools :: Console, defect)

Description

[Alexandre Poirot [:ochameau]]

STR:

• enable fission
• load http://janodvarko.cz/tests/fission/case1/index.html
• open devtools, on the webconsole
• reload the webpage

ER:
See Simple log from odvarko.com in the console

AR:
Simple log from odvarko.com is not visible.

The console message is visible when you open the devtools, but not when you reload the page. We only miss messages that are done early when the remote iframe just started loading.

Comment 1

[Alexandre Poirot [:ochameau]]

This regressed by bug 1657310.
This delayed the call of ResourceWatcher.onTargetAvailable which is registering the resource listeners as early as the target is available, so that we shouldn't miss any early resource.
https://searchfox.org/mozilla-central/source/devtools/shared/resources/resource-watcher.js#247-262

But now, onTargetAvailable are called slightly later as we attach the thread actor first:
https://searchfox.org/mozilla-central/source/devtools/shared/resources/target-list.js#183-198
And that makes us miss the resource-available-form events.

In addition to that, there is a few additional await in ResourceWatcher.onTargetAvailable which may also make us miss some events
and the code in the server may also be better written to ensure notifying about the target before listening to resources.

Comment 2

[Alexandre Poirot [:ochameau]]

Also, this highlights the lack of test to cover such feature. I thought ResourceWatcher tests would cover that, but it looks like they don't.

Comment 3

[Alexandre Poirot [:ochameau]]

Flagging as reserve as we are running out of mvp slots. But this should hopefully be fixed soon, for the nightly experiment.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1657310

Comment 5

[Chris Peterson [:cpeterson]]

Tracking dt-fission-m2-reserve bugs for Fission Beta milestone (M7).

Comment 6

[Alexandre Poirot [:ochameau]]

This have been fixed by bug 1667362.