1691836 - Inspector Markup view might not be fully initialized after using Toolbox::selectTool

Status: RESOLVED FIXED

(DevTools :: Inspector, task, P3)

Description

[Julian Descottes [:jdescottes]]

See https://phabricator.services.mozilla.com/D104206#inline-585178 for more context.

When the inspector panel is returned by a call to await toolbox.selectTool("inspector") it is not fully loaded yet. The markup view frame has loaded its document, but the inspector will still be expanding the selected node, etc...

To workaround this, tests wait for an additional inspector-updated event (see https://searchfox.org/mozilla-central/search?q=inspector._updateProgress&path=test)

We could try to change the inspector init logic to wait until the "reloaded" event instead of the "markuploaded" event to resolve the initial open call.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1691836 - [devtools] Wait for the initialization of markupview in inspector open()

We used to only wait until the markup view document was loaded, via the markuploaded event.
This lead tests to wait for additional events to make sure the inspector markup view could
be interacted with.

Here we wait for a new dedicated internal promise instead of an event.
This promise is only resolved once the inspector has initialized the markup view.

Test helpers are updated to stop listening for additional events.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6d7ff75c6ab8
[devtools] Wait for the initialization of markupview in inspector open() r=nchevobbe

Comment 3

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/6d7ff75c6ab8