From d484c5ab497c0ad43c597bead2196ac71107fcfb Mon Sep 17 00:00:00 2001
From: Yemel Jardi <angel.jardi@gmail.com>
Date: Thu, 17 Jul 2014 22:35:24 -0300
Subject: [PATCH] Replace iframe for window.open _blank

---
 index.html                   | 1 -
 js/controllers/backup.js     | 2 +-
 js/controllers/header.js     | 4 ++--
 js/services/backupService.js | 8 ++++----
 4 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/index.html b/index.html
index 544415228..13c84811b 100644
--- a/index.html
+++ b/index.html
@@ -183,7 +183,6 @@
     </div>
   </div>
 
-  <iframe ng-src="{{mobileBackupURI}}" style="width: 0px; height: 0px; margin:0px; border: 0px"></iframe>
   <div id="footer" data-ng-controller="FooterController" ng-class="{'footer-home': !$root.wallet || !$root.wallet.isReady()}">
     <link rel="stylesheet" ng-href="{{theme}}">
     <div ng-show="!$root.wallet">
diff --git a/js/controllers/backup.js b/js/controllers/backup.js
index 5555455bd..509163b9d 100644
--- a/js/controllers/backup.js
+++ b/js/controllers/backup.js
@@ -3,7 +3,7 @@
 angular.module('copayApp.controllers').controller('BackupController',
   function($scope, $rootScope, backupService, walletFactory, controllerUtils) {
     $scope.download = function() {
-      backupService.download($rootScope.wallet, $rootScope);
+      backupService.download($rootScope.wallet);
     };
 
     $scope.deleteWallet = function() {
diff --git a/js/controllers/header.js b/js/controllers/header.js
index 63c26b008..3da4cc23e 100644
--- a/js/controllers/header.js
+++ b/js/controllers/header.js
@@ -90,12 +90,12 @@ angular.module('copayApp.controllers').controller('HeaderController',
     $scope.backup = function() {
       var w = $rootScope.wallet;
       w.setBackupReady();
-      backupService.download(w, $rootScope);
+      backupService.download(w);
     };
 
     $scope.dowloadBackup = function() {
       var w = $rootScope.wallet;
-      backupService.download(w, $rootScope);
+      backupService.download(w);
     }
 
     $scope.getVideoURL = function(copayer) {
diff --git a/js/services/backupService.js b/js/services/backupService.js
index f059cb907..41fbe559f 100644
--- a/js/services/backupService.js
+++ b/js/services/backupService.js
@@ -9,7 +9,7 @@ BackupService.prototype.getName = function(wallet) {
   return (wallet.name ? (wallet.name + '-') : '') + wallet.id;
 };
 
-BackupService.prototype.download = function(wallet, scope) {
+BackupService.prototype.download = function(wallet) {
   var ew = wallet.toEncryptedObj();
   var partial = !wallet.publicKeyRing.isComplete();
   var walletName = this.getName(wallet) + (partial ? '-Partial' : '');
@@ -30,13 +30,13 @@ BackupService.prototype.download = function(wallet, scope) {
   }
 
   // throw an email intent if we are in the mobile version
-  if (window.xwalk && scope) {
+  if (window.xwalk) {
     var name = wallet.name ? wallet.name + ' ' : '';
     var partial = partial ? 'Partial ' : '';
     var subject = 'Copay - ' + name + 'Wallet ' + partial + 'Backup';
     var body = 'This is the encrypted backup of the wallet ' + wallet.id + ':\n\n' + ew;
-    scope.mobileBackupURI = encodeURI('mailto:?subject=' + subject + '&body=' + body);
-    return;
+    var mailURL = encodeURI('mailto:?subject=' + subject + '&body=' + body);
+    return window.open(mailURL,'_blank');
   }
 
   // otherwise lean on the browser implementation