1702078 - a11y panel should emit ready event after rendering the UI

Status: RESOLVED FIXED

(DevTools :: Accessibility Tools, task, P3)

Description

[Julian Descottes [:jdescottes]]

Spawned from Bug 1699619.

See https://phabricator.services.mozilla.com/D110247

a11y panel open should be fully finished before resolving, but doing that introduces a small regression regarding the "focused" state on the main content document.

We should discuss if this is acceptable or if we need to find a workaround.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1702078 - [devtools] a11y panel should emit ready event after rendering the UI

With the previous implementation, the toolbox would resolve the panel before all the initialization data had been retrieved.
This could lead to issues if we tried to destroy the toolbox right after the panel got selected.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3c11553b2e79
[devtools] a11y panel should emit ready event after rendering the UI r=ochameau

Comment 3

[Atila Butkovits]

Backed out for damp failure in accessibility/cold-open.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/681364ef246014ae15b4c880c5ce9809cb41d6fb

Failures started here: https://treeherder.mozilla.org/jobs?repo=autoland&searchStr=damp&revision=cc986541595b2cbad3631854db03056be997bbab

Failure log: https://treeherder.mozilla.org/logviewer?job_id=335734073&repo=autoland&lineNumber=1066

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0bd7c7103be0
[devtools] a11y panel should emit ready event after rendering the UI r=ochameau,perftest-reviewers

Comment 5

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/0bd7c7103be0