Closed Bug 1835779 Opened 4 months ago Closed 4 months ago

Render the Blackbox Lines component only when the required

Categories

(DevTools :: Debugger, defect, P3)

Product:
DevTools
Component:
Debugger
Type:
defect

Tracking

(firefox-esr102 unaffected, firefox113 unaffected, firefox114 unaffected, firefox115 fixed)

Status:
RESOLVED FIXED
Milestone:
115 Branch
Tracking Flags:
Tracking Status
firefox-esr102 --- unaffected
firefox113 --- unaffected
firefox114 --- unaffected
firefox115 --- fixed

People

(Reporter: bomsy, Assigned: bomsy)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1835779 - [devtools] Load the Blackbox lines component only when needed r=#devtools-reviewers
48 bytes, text/x-phabricator-request
Details | Review

Only render the Blackbox Lines component

  • If the source is on the sourcemaps ignore list
  • When the source is fully or partially blackboxed

https://searchfox.org/mozilla-central/source/devtools/client/debugger/src/components/Editor/index.js#691

Set release status flags based on info from the regressing bug 1832019

status-firefox113: --- → unaffected
status-firefox114: --- → unaffected
status-firefox115: --- → affected
status-firefox-esr102: --- → unaffected
Assignee: nobody → hmanilla
Attachment #9336520 - Attachment description: WIP: Bug 1835779 - [devtools] Load the Blackbox lines component only when needed → Bug 1835779 - [devtools] Load the Blackbox lines component only when needed r=#devtools-reviewers
Status: NEW → ASSIGNED
Severity: -- → S3
Priority: -- → P3
Pushed by hmanilla@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cbc2c6fc648b
[devtools] Load the Blackbox lines component only when needed r=ochameau

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

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
status-firefox115: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch

== Change summary for alert #38553 (as of Sat, 03 Jun 2023 23:41:57 GMT) ==

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
11% damp custom.jsdebugger.pretty-print.DAMP linux1804-64-shippable-qr e10s fission stylo webrender-sw 460.41 -> 408.87
8% damp custom.jsdebugger.pretty-print.DAMP linux1804-64-shippable-qr e10s fission stylo webrender 447.10 -> 410.23
7% damp custom.jsdebugger.pretty-print.DAMP windows10-64-shippable-qr e10s fission stylo webrender-sw 324.00 -> 300.34
7% damp custom.jsdebugger.pretty-print.DAMP windows10-64-shippable-qr e10s fission stylo webrender 321.57 -> 300.38
6% damp custom.jsdebugger.pretty-print.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender 325.17 -> 304.63

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=38553

You need to log in before you can comment on or make changes to this bug.