1685413 - Re-enable devtools/client/dom/test/browser_dom_fission_target_switching.js test for Fission

Status: RESOLVED FIXED

(DevTools :: General, defect, P3)

Description

[Chris Peterson [:cpeterson]]

This devtools/client/dom/test/browser_dom_fission_target_switching.js test is currently skipped for Fission (by bug 1675020):

[browser_dom_fission_target_switching.js]
skip-if = fission # Disable frequent fission intermittents Bug 1675020

https://searchfox.org/mozilla-central/rev/ef900cd2258d4c5d968093f612f807d96e6e7c98/devtools/client/dom/test/browser.ini#14-15

Is this test still failing with Fission?
Does fixing and re-enabling this test need to block shipping Fission MVP?

Comment 1

[Chris Peterson [:cpeterson]]

Tracking DevTools Fission bugs for Fission M7 (blocking Beta experiment).

Comment 2

[Julian Descottes [:jdescottes]]

We can reenable this test on Fission and see if the existing intermittent Bug 1615737 shows a spike of failures.

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1685413 - [devtools] Re-enable devtools/client/dom/test/browser_dom_fission_target_switching.js test for Fission. r=ladybenko.

The test used to fail after navigating because the expected property
wasn't displayed.
Here we're trying to work around that by:

• using the navigateTo helper function, which takes care of waiting for the right events
• awaiting until the property is displayed (since it wasn't before), just in case the
  rendering takes more time than expected.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/655b0c06edc6
[devtools] Re-enable  devtools/client/dom/test/browser_dom_fission_target_switching.js test for Fission. r=ladybenko.

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/655b0c06edc6