Closed
Bug 1791973
Opened 2 years ago
Closed 1 year ago
Always retrieve the watcher front via commands.watcherFront
Categories
(DevTools :: Framework, enhancement)
DevTools
Framework
Tracking
(firefox110 fixed)
RESOLVED
FIXED
110 Branch
Tracking | Status | |
---|---|---|
firefox110 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
While working on bug 1785106, I saw that there was a couple of leftovers still manually fetching the watcher front via descriptorFront.getWatcher()
.
That, whereas the commands object already does that early and then exposes it to all commands and also via commands.watcherFront
.
Assignee | ||
Updated•2 years ago
|
Assignee: nobody → poirot.alex
Assignee | ||
Comment 1•1 year ago
|
||
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d43a5e6878f4 [devtools] Always retrieve the Watcher Front from commands. r=jdescottes
Comment 3•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox110:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Comment 4•1 year ago
|
||
This seems to have slightly regressed complicated.jsdebugger.close. It's not clear why, but the pushlog really points to this single changeset.
== Change summary for alert #36556 (as of Fri, 23 Dec 2022 23:21:36 GMT) ==
Regressions:
Ratio | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|
17% | damp complicated.jsdebugger.close.DAMP | windows10-64-shippable-qr | e10s fission stylo webrender-sw | 27.91 -> 32.79 |
16% | damp complicated.jsdebugger.close.DAMP | linux1804-64-shippable-qr | e10s fission stylo webrender | 26.13 -> 30.19 |
13% | damp complicated.jsdebugger.close.DAMP | windows10-64-shippable-qr | e10s fission stylo webrender | 28.08 -> 31.67 |
8% | damp complicated.jsdebugger.close.DAMP | macosx1015-64-shippable-qr | e10s fission stylo webrender | 22.55 -> 24.37 |
7% | damp custom.jsdebugger.close.DAMP | windows10-64-shippable-qr | e10s fission stylo webrender-sw | 31.94 -> 34.30 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=36556
You need to log in
before you can comment on or make changes to this bug.
Description
•