Closed Bug 860395 Opened 7 years ago Closed 7 years ago

[ImageContainedChild] leaking SharedImage objects in ImageBridgeCopyAndSendTask::Run()


(Firefox OS Graveyard :: General, defect)

Gonk (Firefox OS)
Not set


(blocking-b2g:tef+, firefox20 unaffected, firefox21 unaffected, firefox22 unaffected, firefox23 unaffected, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 fixed)

B2G C4 (2jan on)
blocking-b2g tef+
Tracking Status
firefox20 --- unaffected
firefox21 --- unaffected
firefox22 --- unaffected
firefox23 --- unaffected
b2g18 --- fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- fixed


(Reporter: mikeh, Assigned: mikeh)



(Whiteboard: [MemShrink], QARegressExclude)


(1 file)

Moving patch from bug 846903 to its own bug; propagating flags, setting as blocker.
Attachment #735874 - Flags: review?(nical.bugzilla)
Comment on attachment 735874 [details] [diff] [review]
delete SharedImage object in ImageBridgeCopyAndSendTask::Run() to prevent leak

Carrying over r=nical from bug 846903 attachment 735850 [details] [diff] [review].
Attachment #735874 - Flags: review?(nical.bugzilla) → review+
blocking-b2g: tef? → tef+
jlebar, if you (or someone else) can tef+ this, I'll land it on b2g18 this afternoon.
blocking-b2g: tef+ → tef?
Flags: needinfo?(justin.lebar+bug)
No longer blocks: 846903
Whiteboard: [MemShrink]
Blocks: 846903
I tef+'ed it, but then bugzilla reverted it along with your comment!
Flags: needinfo?(justin.lebar+bug)
blocking-b2g: tef? → tef+

(Change _only_ applies to b2g18 branches.)
Closed: 7 years ago
Resolution: --- → FIXED
Can you please provide steps to verify this fix - as we will blackbox test from the UI?
Flags: needinfo?
(In reply to croesch from comment #6)
> Can you please provide steps to verify this fix - as we will blackbox test
> from the UI?

There is no way to blackbox this from the UI.  The symptoms of the leak this fixes can only really be seen from the adb-shell command line.
Flags: needinfo?
Based on Comment 7, marking as QARegressExclude.
Whiteboard: [MemShrink] → [MemShrink], QARegressExclude
You need to log in before you can comment on or make changes to this bug.