Closed Bug 1266466 Opened 5 years ago Closed 5 years ago

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

Categories

(Core :: Widget: Android, defect)

48 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: kats, Assigned: kats)

Details

Attachments

(1 file)

Attached 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+
https://hg.mozilla.org/mozilla-central/rev/d2670d2a634a
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.