1725546 - Fix intermittent inspector tests failures with SSTS enabled

Status: RESOLVED FIXED

(DevTools :: Inspector, defect, P5)

Description

[Hubert Boma Manilla (:bomsy)]

A couple of inspector tests are failing intermittently due to grid panel updates triggered by reflow updates.

• browser_rules_grid-highlighter-on-navigate.js
• browser_rules_media-queries_reload.js

Example failure summary

TEST-UNEXPECTED-FAIL | devtools/client/inspector/rules/test/browser_rules_grid-highlighter-on-navigate.js | A promise chain failed to handle a rejection: Connection closed, pending request to server0.conn19.windowGlobal2147483658/layout32, type getGrids failed

Comment 1

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

Thanks for the report! Can you please also attach a list of the tests that are failing?

Edit: ah, the first comment was just edited :-)

Comment 2

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1725546 - [devtools] Disable grid panel updates triggered by reflow r=nchevobbe

Depends on D122492

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

Note that we were seeing this kind of failures before server side targets and we already had workaround for them (e.g. https://searchfox.org/mozilla-central/rev/8f08c21f093be1c1c42438697f8bca67af94fc77/devtools/client/inspector/grids/test/browser_grids_grid-outline-highlight-cell.js#31)

This bug should be about having a helper we can call to either:

• disable grid update on reflows (i.e. what's already existing in our tests, but centralized)
• Use PromiseTestUtils.allowMatchingRejectionsGlobally to ignore those Promise rejections, something like:

function silenceGetGridsOnReflowPromiseRejection() {
  const { PromiseTestUtils } = ChromeUtils.import(
    "resource://testing-common/PromiseTestUtils.jsm"
  );
  const removeRejectionException = PromiseTestUtils.allowMatchingRejectionsGlobally(
    /Connection closed.*getGrids failed/
  );
  registerCleanupFunction(removeRejectionException);
}

At the moment, any call to PromiseTestUtils.allowMatchingRejectionsGlobally will impact the whole test suite and there's no mechanism to remove an entry from it. We might want to do that as part of Bug 1655422.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e4978141f75e
[devtools] Disable grid panel updates triggered by reflow r=nchevobbe

Comment 5

[Hubert Boma Manilla (:bomsy)]

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #3)

This bug should be about having a helper we can call to either:

• disable grid update on reflows (i.e. what's already existing in our tests, but centralized)
• Use PromiseTestUtils.allowMatchingRejectionsGlobally to ignore those Promise rejections, something like:

Just to mention, Bug 1725350 was for adding the helper and migrating current tests to using the helper.

At the moment, any call to PromiseTestUtils.allowMatchingRejectionsGlobally will impact the whole test suite and there's no mechanism to remove > an entry from it. We might want to do that as part of Bug 1655422.

Added a bug for it. See Bug 1725565

Thanks

Comment 6

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/e4978141f75e