Closed Bug 1383333 Opened 3 years ago Closed 3 years ago

Provide easy access to the currently selected tab

Categories

(Core :: DOM: Content Processes, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: billm, Assigned: billm)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
I've decided to make a simplifying assumption for the initial Quantum DOM patches: if only one tab is in the foreground, then we'll label vsyncs correctly for that tab and also prioritize runnables for that tab. If multiple tabs (from multiple windows) are in the foreground, then you won't get any benefit from Quantum DOM. This shouldn't be too hard to fix in the future, but it's a pain to deal with right now.

This patch provides an easy accessor for the foreground TabChild (if there is only one). I imagine other people might find this useful as well for various performance optimizations.
Attachment #8888993 - Flags: review?(dvander)
Attachment #8888993 - Flags: review?(dvander) → review+
Summary: Provide easy access to the currently selected tab (when there is only one) → Provide easy access to the currently selected tab
Attached patch patch v2Splinter Review
I realized that we do need to keep track of multiple active tabs. When tab switching, for example, two tabs can be active at once. And that's specifically a case where I would like the Quantum DOM scheduler to be effective.

David is on PTO, so asking Kan-Ru for review.
Attachment #8888993 - Attachment is obsolete: true
Attachment #8899011 - Flags: review?(kchen)
Attachment #8899011 - Flags: review?(kchen) → review+
Pushed by wmccloskey@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7f426b4432ff
Keep track of foreground TabChildren (r=kanru)
https://hg.mozilla.org/mozilla-central/rev/7f426b4432ff
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.