Closed Bug 414325 Opened 12 years ago Closed 12 years ago
Utils for extension/theme manager downloads
Good reusable code, lets us drop some of our code and strings.
Assignee: nobody → dtownsend
Status: NEW → ASSIGNED
Attachment #299786 - Flags: review?(robert.bugzilla)
I believe _replaceInsert was only used by _formatKBytes, so up to you if you want to keep it around if something in the future might want it.
Yeah well spotted, I don't think we need that anymore.
Comment on attachment 299800 [details] [diff] [review] patch rev 2 Nice!
Attachment #299800 - Flags: review?(robert.bugzilla) → review+
Comment on attachment 299800 [details] [diff] [review] patch rev 2 This is a code saving, switching to shared code for this.
Attachment #299800 - Flags: approval1.9?
Comment on attachment 299800 [details] [diff] [review] patch rev 2 a1.9+=damons
Attachment #299800 - Flags: approval1.9? → approval1.9+
Checking in toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties; /cvsroot/mozilla/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties,v <-- extensions.properties new revision: 1.49; previous revision: 1.48 done Checking in toolkit/mozapps/extensions/content/extensions.js; /cvsroot/mozilla/toolkit/mozapps/extensions/content/extensions.js,v <-- extensions.js new revision: 1.158; previous revision: 1.157 done
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [has patch]
Target Milestone: --- → Firefox 3 beta3
You need to log in before you can comment on or make changes to this bug.