Closed
Bug 1139005
Opened 10 years ago
Closed 10 years ago
Need to check the return value of AutoJSAPI.init in canvas EncodeCallback
Categories
(Core :: Audio/Video, defect)
Tracking
()
People
(Reporter: billm, Assigned: billm)
References
Details
Attachments
(1 file)
(deleted),
patch
|
ayang
:
review+
bajaj
:
approval-mozilla-b2g37+
|
Details | Diff | Splinter Review |
This was causing problems if the callback is called after the window has already been torn down.
Attachment #8572033 -
Flags: review?(ayang)
Comment 1•10 years ago
|
||
Comment on attachment 8572033 [details] [diff] [review]
jsapi
Review of attachment 8572033 [details] [diff] [review]:
-----------------------------------------------------------------
Thank you!
BTW, how do you reproduce this bug? I doubt this bug probably be something like https://crash-stats.mozilla.com/report/index/981be8cb-79ef-4f04-b44d-bc9b22150115.
Attachment #8572033 -
Flags: review?(ayang) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9e83a9eba19f
I had a patch that made this happen more often. It was just a coincidence.
Comment 4•10 years ago
|
||
Comment on attachment 8572033 [details] [diff] [review]
jsapi
NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 1139005
User impact if declined: gallery crash in low memory device (256mb)
Testing completed: local test is fine.
Risk to taking this patch (and alternatives if risky): low
String or UUID changes made by this patch: na
Attachment #8572033 -
Flags: approval-mozilla-b2g37?
Comment 5•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Updated•10 years ago
|
Attachment #8572033 -
Flags: approval-mozilla-b2g37? → approval-mozilla-b2g37+
Comment 6•10 years ago
|
||
status-b2g-v2.2:
--- → fixed
status-b2g-master:
--- → fixed
status-firefox37:
--- → wontfix
status-firefox38:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•