m/metamask-extension
1
0
Fork 0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-12-23 09:52:26 +01:00
Code

Remove unused background API methods (#12731)

Browse Source 
These background API methods were not used anywhere in the UI. One of
them was called in `actions.js` by a function that itself was never
called, so it have been removed. Additionally, one unused `actions.js`
function was found and removed as well.

`setAdvancedGasFee` is the only unused background method that remains.
It was recently added and will be used in the near future.
This commit is contained in:
Mark Stacey 2021-11-17 15:03:36 -03:30 committed by GitHub
parent ecceb4d6da
commit cce2dda8e4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 0 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
app/scripts/metamask-controller.js
View File

@ -1008,9 +1008,7 @@ export default class MetamaskController extends EventEmitter {
), ),
createCancelTransaction: nodeify(this.createCancelTransaction, this), createCancelTransaction: nodeify(this.createCancelTransaction, this),
createSpeedUpTransaction: nodeify(this.createSpeedUpTransaction, this), createSpeedUpTransaction: nodeify(this.createSpeedUpTransaction, this),
isNonceTaken: nodeify(txController.isNonceTaken, txController),
estimateGas: nodeify(this.estimateGas, this), estimateGas: nodeify(this.estimateGas, this),
getPendingNonce: nodeify(this.getPendingNonce, this),
getNextNonce: nodeify(this.getNextNonce, this), getNextNonce: nodeify(this.getNextNonce, this),
addUnapprovedTransaction: nodeify( addUnapprovedTransaction: nodeify(
txController.addUnapprovedTransaction, txController.addUnapprovedTransaction,
@ -1094,13 +1092,6 @@ export default class MetamaskController extends EventEmitter {
permissionsController.approvePermissionsRequest, permissionsController.approvePermissionsRequest,
permissionsController, permissionsController,
), ),
clearPermissions: permissionsController.clearPermissions.bind(
permissionsController,
),
getApprovedAccounts: nodeify(
permissionsController.getAccounts,
permissionsController,
),
rejectPermissionsRequest: nodeify( rejectPermissionsRequest: nodeify(
permissionsController.rejectPermissionsRequest, permissionsController.rejectPermissionsRequest,
permissionsController, permissionsController,

19
ui/store/actions.js
View File

@ -1356,12 +1356,6 @@ export function acceptWatchAsset(suggestedAssetID) {
}; };
} }
export function addKnownMethodData(fourBytePrefix, methodData) {
return () => {
background.addKnownMethodData(fourBytePrefix, methodData);
};
}
export function clearPendingTokens() { export function clearPendingTokens() {
return { return {
type: actionConstants.CLEAR_PENDING_TOKENS, type: actionConstants.CLEAR_PENDING_TOKENS,
@ -2403,19 +2397,6 @@ export function removePermissionsFor(domains) {
}; };
} }
/**
* Clears all permissions for all domains.
*/
export function clearPermissions() {
return (dispatch) => {
background.clearPermissions((err) => {
if (err) {
dispatch(displayWarning(err.message));
}
});
};
}
// Pending Approvals // Pending Approvals
/** /**