Closed Bug 1257936 Opened 8 years ago Closed 8 years ago

LeakCanary: (TabsPanel.mAddTab) BrowserApp has leaked 1.0MB

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set
normal

Tracking

(firefox48 fixed)

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- fixed

People

(Reporter: sebastian, Assigned: sebastian)

References

Details

Attachments

(3 files)

Attached file leak-report.txt
> * org.mozilla.gecko.BrowserApp has leaked:
> * GC ROOT static org.mozilla.gecko.tabs.TabsPanel.mAddTab
> * references android.widget.ImageButton.mContext
> * leaks org.mozilla.gecko.BrowserApp instance
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Comment on attachment 8732318 [details]
MozReview Request: Bug 1257936 - TabsPanel: Do not hold static references to views. r?ahunt

https://reviewboard.mozilla.org/r/41093/#review37713

WOW!
Attachment #8732318 - Flags: review?(ahunt) → review+
https://hg.mozilla.org/mozilla-central/rev/ab15d48afbb2
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.