ui - remove unused currentDomain prop

feature/default_network_editable
kumavis 8 years ago
parent 5126ddaf0c
commit 8a5007a396
  1. 19
      app/scripts/popup.js
  2. 1
      ui/app/accounts/index.js
  3. 1
      ui/app/reducers/app.js
  4. 1
      ui/app/reducers/metamask.js
  5. 4
      ui/index.js

@ -1,4 +1,3 @@
const url = require('url')
const EventEmitter = require('events').EventEmitter const EventEmitter = require('events').EventEmitter
const async = require('async') const async = require('async')
const Dnode = require('dnode') const Dnode = require('dnode')
@ -18,7 +17,6 @@ var css = MetaMaskUiCss()
injectCss(css) injectCss(css)
async.parallel({ async.parallel({
currentDomain: getCurrentDomain,
accountManager: connectToAccountManager, accountManager: connectToAccountManager,
}, setupApp) }, setupApp)
@ -46,6 +44,8 @@ function setupWeb3Connection (stream) {
} }
function setupControllerConnection (stream, cb) { function setupControllerConnection (stream, cb) {
// this is a really sneaky way of adding EventEmitter api
// to a bi-directional dnode instance
var eventEmitter = new EventEmitter() var eventEmitter = new EventEmitter()
var background = Dnode({ var background = Dnode({
sendUpdate: function (state) { sendUpdate: function (state) {
@ -60,20 +60,6 @@ function setupControllerConnection (stream, cb) {
}) })
} }
function getCurrentDomain (cb) {
const unknown = '<unknown>'
if (!extension.tabs) return cb(null, unknown)
extension.tabs.query({active: true, currentWindow: true}, function (results) {
var activeTab = results[0]
var currentUrl = activeTab && activeTab.url
var currentDomain = url.parse(currentUrl).host
if (!currentUrl) {
return cb(null, unknown)
}
cb(null, currentDomain)
})
}
function setupApp (err, opts) { function setupApp (err, opts) {
if (err) { if (err) {
alert(err.stack) alert(err.stack)
@ -85,7 +71,6 @@ function setupApp (err, opts) {
MetaMaskUi({ MetaMaskUi({
container: container, container: container,
accountManager: opts.accountManager, accountManager: opts.accountManager,
currentDomain: opts.currentDomain,
networkVersion: opts.networkVersion, networkVersion: opts.networkVersion,
}) })
} }

@ -20,7 +20,6 @@ function mapStateToProps (state) {
identities: state.metamask.identities, identities: state.metamask.identities,
unconfTxs: state.metamask.unconfTxs, unconfTxs: state.metamask.unconfTxs,
selectedAddress: state.metamask.selectedAddress, selectedAddress: state.metamask.selectedAddress,
currentDomain: state.appState.currentDomain,
scrollToBottom: state.appState.scrollToBottom, scrollToBottom: state.appState.scrollToBottom,
pending, pending,
} }

@ -39,7 +39,6 @@ function reduceApp (state, action) {
accountDetail: { accountDetail: {
subview: 'transactions', subview: 'transactions',
}, },
currentDomain: 'example.com',
transForward: true, // Used to render transition direction transForward: true, // Used to render transition direction
isLoading: false, // Used to display loading indicator isLoading: false, // Used to display loading indicator
warning: null, // Used to display error text warning: null, // Used to display error text

@ -11,7 +11,6 @@ function reduceMetamask (state, action) {
isInitialized: false, isInitialized: false,
isUnlocked: false, isUnlocked: false,
isEthConfirmed: false, isEthConfirmed: false,
currentDomain: 'example.com',
rpcTarget: 'https://rawtestrpc.metamask.io/', rpcTarget: 'https://rawtestrpc.metamask.io/',
identities: {}, identities: {},
unconfTxs: {}, unconfTxs: {},

@ -25,9 +25,7 @@ function startApp (metamaskState, accountManager, opts) {
metamask: metamaskState, metamask: metamaskState,
// appState represents the current tab's popup state // appState represents the current tab's popup state
appState: { appState: {},
currentDomain: opts.currentDomain,
},
// Which blockchain we are using: // Which blockchain we are using:
networkVersion: opts.networkVersion, networkVersion: opts.networkVersion,

Loading…
Cancel
Save