Closed Bug 1589744 Opened 5 years ago Closed 5 years ago

Optimize hidden tabs

Categories

(DevTools :: Debugger, defect, P2)

defect

Tracking

(firefox72 fixed)

RESOLVED FIXED
Firefox 72
Tracking Status
firefox72 --- fixed

People

(Reporter: mozilla-bugzilla, Assigned: mozilla-bugzilla)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:71.0) Gecko/20100101 Firefox/71.0

Steps to reproduce:

  • Load a Webpack-powered website with ~60 JavaScript bundles (that are source-mapped to ~4000 files)
  • Open all the 60 bundles as tabs in the Debugger
  • Close the website
  • Open the website in a new tab (with devtools closed)
  • Wait for the website to finish loading
  • Open the devtools

Actual results:

The debugger hangs for 20-25 seconds.

Expected results:

The debugger should be snappier.

They only need to be updated from componentDidUpdate if the open tabs have changed.

Before: https://perfht.ml/32w6iwg
After: https://perfht.ml/2Bqk7R0
Diff: https://perfht.ml/2pvNyhW

Background churn in the debugger is a long-standing issue, so making this P2.

Assignee: nobody → sorin.davidoi
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P2
Attachment #9102610 - Attachment description: perf(debugger/editor/tabs): Don't update hidden tabs when not necessary → Bug 1589744: Don't update hidden tabs when not necessary r=davidwalsh
Pushed by dwalsh@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/86bee6f6a880 Don't update hidden tabs when not necessary r=davidwalsh
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: