Closed Bug 1335730 Opened 5 years ago Closed 5 years ago

FinderHighlighter.jsm leaks about:newtab

Categories

(Toolkit :: Find Toolbar, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla54
Tracking Status
firefox54 --- verified
firefox55 --- verified

People

(Reporter: bkelly, Assigned: mikedeboer)

References

Details

(Whiteboard: [MemShrink])

Attachments

(1 file)

STR:

1) open new browser with findbar.modalHighlight set to true
2) open about:memory in a tab
3) hit ctrl-t to open a new tab (it should show about:newtab by default)
4) on about:newtab hit ctr-f and search for any random string
5) close the about:newtab tab
6) go to your about:memory tab, minimize, and measure
7) observe a detached about:newtab window

The GC/CC logs show this is being held alive by FinderHighlighter.jsm's gWindows Map:

via persistent-Object :
000001EA4691DE20 [BackstagePass 000001EA33DEFC40]
    --[**UNKNOWN SLOT 128**]--> 000001EA352158B0 [LexicalEnvironment <no private>]
    --[gWindows]--> 000001EA3E331A60 [Map 000001EA63F93E80]
    --[key]--> 000001EA38BDE180 [Proxy <no private>]
    --[private]--> 000001EA3B93D0E0 [Proxy <no private>]
    --[private]--> 000001EA3DB78100 [Window <no private>]


Mike, can you take a look at this?
Flags: needinfo?(mdeboer)
Whiteboard: [MemShrink]
Yup! Thanks for filing.
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Flags: needinfo?(mdeboer)
Comment on attachment 8834004 [details]
Bug 1335730 - delete the map entry when a window unloads and is tracked by the FinderHighlighter.

https://reviewboard.mozilla.org/r/110116/#review111126
Attachment #8834004 - Flags: review?(jaws) → review+
Pushed by mdeboer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9a087f3a3767
delete the map entry when a window unloads and is tracked by the FinderHighlighter. r=jaws
https://hg.mozilla.org/mozilla-central/rev/9a087f3a3767
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Verified fixed using the latest Nightly 55.0a1(2017-04-25) and Nightly 54.0a1 (2017-02-11), with the preference "findbar.modalHighlight" set to true on  Windows 10 x64, Ubuntu 16.04 and Mac OS X 10.12.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.