RequestPolicy/tractest — Issue #228: Removal of Unused Functions
Author: jsamuel
Created: 2011-12-22 02:48:03+00:00
State: open
Labels: extension

There appear to be at least 12 (!) functions that are not currently used. Recommend removal of these functions:

''./src/components/requestpolicyService.js''

_combinedOriginToDestinationIdentifierHasOrigin
_combinedOriginToDestinationIdentifierHasDestination
_dumpRejectedRequests
_dumpAllowedRequests

''./src/content/menu.js''

_removeExtraSubmenuSeparators
_disableMenuIfEmpty

''./src/content/overlay.js''

_contextMenuOnPopupHidden
_performRedirect

''./src/modules/DomainUtil.jsm:''

identifierIsInUri
stripWww
sameHostIgnoreWww
destinationIsSubdomainOfOrigin

Note that there are additional functions that are not referenced in the extension; however, they are implied to be required by the IDL of an XPCOM component/observer or similar.