Closed Bug 471547 Opened 12 years ago Closed 12 years ago
DBUtils getters to avoid possible leaks
we should move PlacesDBUtils getters in the constructor see bug 469972 for the reasoning.
Comment on attachment 354881 [details] [diff] [review] patch v1.0 >+ ////////////////////////////////////////////////////////////////////////////// >+ //// Smart getters Make sure these lines don't go past 80 chars please I'm not sure these were leaking before, but it could be possible. r=sdwilsh
Attachment #354881 - Flags: review?(sdwilsh) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: mozilla1.9.1 → mozilla1.9.2a1
Comment on attachment 354881 [details] [diff] [review] patch v1.0 Low risk, and I'd like to see this land on branch too.
Comment on attachment 354881 [details] [diff] [review] patch v1.0 a191=beltzner
Attachment #354881 - Flags: approval1.9.1? → approval1.9.1+
You need to log in before you can comment on or make changes to this bug.