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

Add comments about legacy streams (#15492)

Browse Source 
Two comments have been added to reference a longer explanation of what
the legacy provider streams are, why we still have them, and why we
want to remove them.
This commit is contained in:
Mark Stacey 2022-08-05 15:22:07 -04:00 committed by GitHub
parent 44f8e9e10e
commit 42c8703f3e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/scripts/contentscript.js
View File

@ -27,6 +27,8 @@ const PHISHING_WARNING_PAGE = 'metamask-phishing-warning-page';
const PHISHING_SAFELIST = 'metamask-phishing-safelist'; const PHISHING_SAFELIST = 'metamask-phishing-safelist';
const PROVIDER = 'metamask-provider'; const PROVIDER = 'metamask-provider';
// For more information about these legacy streams, see here:
// https://github.com/MetaMask/metamask-extension/issues/15491
// TODO:LegacyProvider: Delete // TODO:LegacyProvider: Delete
const LEGACY_CONTENT_SCRIPT = 'contentscript'; const LEGACY_CONTENT_SCRIPT = 'contentscript';
const LEGACY_INPAGE = 'inpage'; const LEGACY_INPAGE = 'inpage';

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

@ -1125,6 +1125,8 @@ export default class MetamaskController extends EventEmitter {
this.setupControllerEventSubscriptions(); this.setupControllerEventSubscriptions();
// For more information about these legacy streams, see here:
// https://github.com/MetaMask/metamask-extension/issues/15491
// TODO:LegacyProvider: Delete // TODO:LegacyProvider: Delete
this.publicConfigStore = this.createPublicConfigStore(); this.publicConfigStore = this.createPublicConfigStore();