Created attachment 498729 [details] [diff] [review] Patch v1 Bug 599969 added syncStatements before mShuttingDown in the header, but after in the constructor.
Attachment #498729 - Flags: review?(sdwilsh)
the removal of mozilla:: from the namespacing is correct, but since all the rest of the file uses mozilla::dom and mozilla::services, I'd not change it just in one place. Probably not worth it changing in all places too, imo.
Comment on attachment 498729 [details] [diff] [review] Patch v1 r=sdwilsh
Created attachment 498747 [details] [diff] [review] Patch for checkin
Attachment #498729 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b9
You need to log in before you can comment on or make changes to this bug.