1422052 - inspector and styleeditor DAMP tests have pending requests after reload step.

Status: NEW

(DevTools :: Framework, enhancement, P3)

Description

[Alexandre Poirot [:ochameau]]

While working on bug 1420233 I realized that inspector and styleeditor were both having pending RDP request after "reload" subtest.

In this try push:
https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=try&originalRevision=6884256b36dc397ff5677c311b2dced7c9782275&newProject=try&newRevision=2ed497c64dffa3fe9fbce0656d765e4caad3155f&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=settle&framework=1
We can see the time it takes to wait for all RDP request completion before closing the toolbox (settle-requests). It is anormaly high for inspector and styleeditor.

I think styleeditor subtest is just wrong, it doesn't wait correctly for panel update.
But for the inspector it may only be related to the element flashing, where we briefly show the highlighter when we select an element.

Ideally, it would be better to assert that panels do not have any unexpected pending request after page reload, so that we can get a mochitest to cover that.
But I don't know how easy that would be to write.