cordova-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From agri...@apache.org
Subject [5/5] git commit: Remove unneccessary replace() in CrxInstaller.js
Date Fri, 21 Mar 2014 06:54:12 GMT
Remove unneccessary replace() in CrxInstaller.js


Project: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/commit/cb1e6a01
Tree: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/tree/cb1e6a01
Diff: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/diff/cb1e6a01

Branch: refs/heads/master
Commit: cb1e6a01b2bb15d83836abe4173c6bfca92f027e
Parents: 936a83a
Author: Andrew Grieve <agrieve@chromium.org>
Authored: Thu Mar 20 23:52:31 2014 -0700
Committer: Andrew Grieve <agrieve@chromium.org>
Committed: Thu Mar 20 23:52:31 2014 -0700

----------------------------------------------------------------------
 www/cdvah/js/CrxInstaller.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-app-harness/blob/cb1e6a01/www/cdvah/js/CrxInstaller.js
----------------------------------------------------------------------
diff --git a/www/cdvah/js/CrxInstaller.js b/www/cdvah/js/CrxInstaller.js
index 50fdf8b..a6fdd5a 100644
--- a/www/cdvah/js/CrxInstaller.js
+++ b/www/cdvah/js/CrxInstaller.js
@@ -20,7 +20,7 @@
 
             // The filename doesn't matter, but it needs to end with .crx for the zip plugin
to unpack
             // it properly. So we always set the filename to package.crx.
-            var crxFile = installPath.replace(/\/$/, '') + '/package.crx';
+            var crxFile = installPath + '/package.crx';
 
             return ResourcesLoader.downloadFromUrl(this.url, crxFile).then(function() {
                 return ResourcesLoader.extractZipFile(crxFile, installPath + '/www');


Mime
View raw message