Closed Bug 933992 Opened 6 years ago Closed 6 years ago
Load Tasks removal incorrectly uses long, not Integer
No description provided.
This will cause sLoadMap to grow endlessly, and -- due to some of the inefficiencies revealed in Bug 931843 -- it'll hold on to every view, every thumbnail, ...
Summary: sLoadMap removal incorrectly uses long, not Integer → sLoadTasks removal incorrectly uses long, not Integer
[Approval Request Comment] Bug caused by (feature/regressing bug #): FaviconsRewrite. User impact if declined: Eventual OOM. Testing completed (on m-c, etc.): Tested locally with logging, verified that sLoadTasks tends towards 0 items. Patch up for review now, will bake on m-c. Risk to taking this patch (and alternatives if risky): Should be none. String or IDL/UUID changes made by this patch: None.
Attachment #826140 - Flags: review?(mark.finkle) → review+
Severity: normal → critical
Target Milestone: --- → Firefox 28
Attachment #826140 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Looks cheerful on fx-team, so: https://hg.mozilla.org/releases/mozilla-aurora/rev/4aca57f111d3
Looks like this merge didn't get recorded. https://hg.mozilla.org/mozilla-central/rev/d6136cbe47d1
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.