Android's nsWindow::Destroy doesn't call the superclass' Destroy

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: kats, Assigned: kats)

Tracking

48 Branch
mozilla48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

Posted patch PatchSplinter Review
For Android I'm pretty sure it's a no-op anyway, but for completeness we should do it. Also deathgripping "this" is a good idea because some of that stuff can release the last ref.

Found these while investigating bug 1264161. Try push including this patch at https://treeherder.mozilla.org/#/jobs?repo=try&revision=05e96ebb3278
Attachment #8743930 - Flags: review?(snorp)
Assignee: nobody → bugmail.mozilla
Attachment #8743930 - Flags: review?(snorp) → review+

Comment 2

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d2670d2a634a
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.