Closed
Bug 859434
Opened 12 years ago
Closed 12 years ago
Close thumbnail cursor in TopSitesView
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox21+ fixed, firefox22+ fixed)
RESOLVED
FIXED
Firefox 23
People
(Reporter: mfinkle, Assigned: mfinkle)
Details
Attachments
(1 file)
1.31 KB,
patch
|
lucasr
:
review+
bajaj
:
approval-mozilla-aurora+
bajaj
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
We never close the cursor used to load and decode a thumbnail bitmap in TopSitesView.java
Attachment #734740 -
Flags: review?(lucasr.at.mozilla)
Updated•12 years ago
|
Attachment #734740 -
Flags: review?(lucasr.at.mozilla) → review+
Assignee | ||
Comment 1•12 years ago
|
||
Comment 2•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 734740 [details] [diff] [review]
patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #):
User impact if declined: Not closing cursors can lead to crashes like bug 760394
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): low risk. simple patch
String or IDL/UUID changes made by this patch: none
Attachment #734740 -
Flags: approval-mozilla-beta?
Attachment #734740 -
Flags: approval-mozilla-aurora?
Comment 4•12 years ago
|
||
Comment on attachment 734740 [details] [diff] [review]
patch
Low risk patch which can help with Bug 760394 , a DB related top-crasher.
Attachment #734740 -
Flags: approval-mozilla-beta?
Attachment #734740 -
Flags: approval-mozilla-beta+
Attachment #734740 -
Flags: approval-mozilla-aurora?
Attachment #734740 -
Flags: approval-mozilla-aurora+
Updated•12 years ago
|
Assignee: nobody → mark.finkle
status-firefox21:
--- → affected
status-firefox22:
--- → affected
tracking-firefox21:
--- → +
tracking-firefox22:
--- → +
Assignee | ||
Comment 5•12 years ago
|
||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•