Closed
Bug 1383333
Opened 7 years ago
Closed 7 years ago
Provide easy access to the currently selected tab
Categories
(Core :: DOM: Content Processes, enhancement)
Core
DOM: Content Processes
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: billm, Assigned: billm)
References
Details
Attachments
(1 file, 1 obsolete file)
3.98 KB,
patch
|
kanru
:
review+
|
Details | Diff | 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+
Assignee | ||
Updated•7 years ago
|
Summary: Provide easy access to the currently selected tab (when there is only one) → Provide easy access to the currently selected tab
Assignee | ||
Comment 1•7 years ago
|
||
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)
Updated•7 years ago
|
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)
Comment 3•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•