Open Bug 2016127 Opened 7 months ago Updated 5 months ago

Remove the event loop tick workaround from CssRuleView.selectElement

Categories

(DevTools :: Inspector: Rules, task, P3)

task

Tracking

(Not tracked)

People

(Reporter: ochameau, Unassigned)

References

Details

Followup from this review comment.

      // This is historical, but unfortunately breaks some tests if removed
      await Promise.resolve(null);

Removing this event loop tick leads to test failures:
https://treeherder.mozilla.org/logviewer?job_id=548099316&repo=try&task=NopHaWkQStWkc8UQZgPZCA.0

[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/rules/test/browser_rules_container-queries.js | Uncaught exception in test - TypeError: can't access property "getBoundingClientRect", aTarget is null
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - synthesizeMouse@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:671:14
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - assertJumpToContainerButton@chrome://mochitests/content/browser/devtools/client/inspector/rules/test/browser_rules_container-queries.js:249:14
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - async*@chrome://mochitests/content/browser/devtools/client/inspector/rules/test/browser_rules_container-queries.js:100:9
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - async*handleTask@chrome://mochikit/content/browser-test.js:1302:26
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - _runTaskBasedTest@chrome://mochikit/content/browser-test.js:1392:18
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - async*Tester_execTest@chrome://mochikit/content/browser-test.js:1539:14
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - nextTest/<@chrome://mochikit/content/browser-test.js:1282:14
[task 2026-02-11T09:50:30.552+00:00] 09:50:30     INFO - SimpleTest.waitForFocus/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:1063:13

Moving most untriaged internal task / enhancements to P3

Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.