Open
Bug 1269935
Opened 8 years ago
Updated 1 year ago
Replace PresShell::DecVisibleCount() with a general map function
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
REOPENED
mozilla49
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: seth, Unassigned)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
6.91 KB,
patch
|
mstange
:
review+
|
Details | Diff | Splinter Review |
PresShell::DecVisibleCount() just iterates over all the frames in a visible frames set and decrements a counter for each one for which visibility tracking is still enabled. We'll soon need to add more code that does this kind of iteration, so it makes sense to replace DecVisibleCount() with a general iterator function that takes a lambda as an argument.
Reporter | ||
Comment 1•8 years ago
|
||
Here's the patch. We'll be making use of this map function a good deal in upcoming patches.
Attachment #8748433 -
Flags: review?(mstange)
Updated•8 years ago
|
Attachment #8748433 -
Flags: review?(mstange) → review+
Reporter | ||
Comment 2•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/45c3308d49c95e37c8d1e4855231baa976a7fe9a Bug 1269935 - Replace PresShell::DecVisibleCount() with a general map function. r=mstange
Comment 3•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/45c3308d49c9
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Comment 4•8 years ago
|
||
I backed this bug out on inbound, I expect the backout to get merged to mozilla-central. I plan to also request uplift to backout on aurora (so this bug would not be landed in any version of Firefox). Bug 1284350 tracks the back out.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 5•2 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.
Assignee: seth.bugzilla → nobody
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•