395311 - EM general code cleanup

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, defect)

Description

[Robert Strong (they/them - no direct email)]

Attachment: patch

Cleanup the EM implementation so it is consistent with other JS components and remove unused code.

Comment 1

[Dave Townsend [:mossop]]

Comment on attachment 280003 [details] [diff] [review]
patch

r=me

Comment 2

[Robert Strong (they/them - no direct email)]

Checked in to trunk

Checking in mozilla/toolkit/mozapps/extensions/src/nsExtensionManager.js.in;
/cvsroot/mozilla/toolkit/mozapps/extensions/src/nsExtensionManager.js.in,v  <--  nsExtensionManager.js.in
new revision: 1.254; previous revision: 1.253
done
Checking in mozilla/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.43; previous revision: 1.42
done

Comment 3

[Rimas Kudelis [use needinfo to ping]]

FYI: the "toolkitName" string was only recently added to extensions.properties (see bug 299716), are you sure you wanted to remove it?

Comment 4

[Dave Townsend [:mossop]]

(In reply to comment #3)
> FYI: the "toolkitName" string was only recently added to extensions.properties
> (see bug 299716), are you sure you wanted to remove it?

The string is no longer in use.