Turn off full screen vs popup a/b test

MetaMask · Oct 21, 2019 · cda9e3f · cda9e3f
1 parent 4f9ea17
commit cda9e3f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 19 deletions.
diff --git a/app/scripts/background.js b/app/scripts/background.js
@@ -233,13 +233,11 @@ function setupController (initState, initLangCode) {
   //
   // MetaMask Controller
   //
-  const { ABTestController = {} } = initState
-  const { abTests = {} } = ABTestController
 
   const controller = new MetamaskController({
     // User confirmation callbacks:
     showUnconfirmedMessage: triggerUi,
-    showUnapprovedTx: abTests.fullScreenVsPopup === 'fullScreen' ? triggerUiInNewTab : triggerUi,
+    showUnapprovedTx: triggerUi,
     openPopup: openPopup,
     closePopup: notificationManager.closePopup.bind(notificationManager),
     // initial state
@@ -443,20 +441,6 @@ function triggerUi () {
   })
 }
 
-/**
- * Opens a new browser tab for user confirmation
- */
-function triggerUiInNewTab () {
-  const tabIdsArray = Object.keys(openMetamaskTabsIDs)
-  if (tabIdsArray.length) {
-    extension.tabs.update(parseInt(tabIdsArray[0], 10), { 'active': true }, () => {
-      extension.tabs.reload(parseInt(tabIdsArray[0], 10))
-    })
-  } else {
-    platform.openExtensionInBrowser()
-  }
-}
-
 /**
  * Opens the browser popup for user confirmation of watchAsset
  * then it waits until user interact with the UI

diff --git a/ui/app/pages/confirm-transaction/confirm-transaction.container.js b/ui/app/pages/confirm-transaction/confirm-transaction.container.js
@@ -36,9 +36,9 @@ const mapStateToProps = (state, ownProps) => {
   const transaction = totalUnconfirmed
     ? unapprovedTxs[id] || unconfirmedTransactions[totalUnconfirmed - 1]
     : {}
-  const { id: transactionId, transactionCategory, origin } = transaction
+  const { id: transactionId, transactionCategory } = transaction
 
-  const trackABTest = origin !== 'MetaMask'
+  const trackABTest = false
 
   return {
     totalUnapprovedCount: totalUnconfirmed,