CB-10441 Add auto tests for InAppBrowser plugin

.gitignore

@@ -12,7 +12,6 @@ Thumbs.db
 *.swp
 *.user
 
-node_modules
 
 
 

.jshintrc

@@ -1,16 +0,0 @@
-{
-    "browser": true
-  , "devel": true
-  , "bitwise": true
-  , "undef": true
-  , "trailing": true
-  , "quotmark": false
-  , "indent": 4
-  , "unused": "vars"
-  , "latedef": "nofunc"
-  , "globals": {
-        "module": false,
-        "exports": false,
-        "require": false
-    }
-}

.travis.yml

@@ -1,4 +0,0 @@
-language: node_js
-sudo: false
-node_js:
-  - "4.2"

CONTRIBUTING.md

@@ -35,30 +35,3 @@ The details are explained there, but the important items are:
  - Run the tests so your patch doesn't break existing functionality.
 
 We look forward to your contributions!
-
-The notes on [Commit Workflow](https://github.com/apache/cordova-coho/blob/master/docs/committer-workflow.md#commit-workflow) can be helpful even if you are not a committer.
-
-## Running plugin tests
-
-* clone and install [cordova-plugin-test-framework](https://github.com/apache/cordova-plugin-test-framework)
-```
-git clone git@github.com:apache/cordova-plugin-test-framework.git
-```
-* edit ```cordova-plugin-test-framework/www/assets/index.html``` and add the following line
-```
-<meta http-equiv="Content-Security-Policy" content="default-src 'self' data: gap: https://ssl.gstatic.com http://cordova.apache.org http://google.co.uk https://google.co.uk 'unsafe-eval'; style-src 'self' 'unsafe-inline'; media-src *">
-```
-* create test project
-```
-cordova create plugintest
-cd plugintest
-cordova platform add android
-cordova plugin add ../cordova-plugin-inappbrowser
-cordova plugin add ../cordova-plugin-inappbrowser/tests
-cordova plugin add ../cordova-plugin-test-framework
-```
-* edit ```config.xml``` and replace ```<content src="index.html" />``` with ```<content src="cdvtests/index.html" />```
-* run application
-```
-cordova run
-```

README.md

@@ -17,8 +17,6 @@
 #         under the License.
 -->
 
-[![Build Status](https://travis-ci.org/apache/cordova-plugin-inappbrowser.svg?branch=master)](https://travis-ci.org/apache/cordova-plugin-inappbrowser)
-
 # cordova-plugin-inappbrowser
 
 This plugin provides a web browser view that displays when calling `cordova.InAppBrowser.open()`.
@@ -56,7 +54,7 @@ Although `window.open` is in the global scope, InAppBrowser is not available unt
         console.log("window.open works well");
     }
 
-Report issues with this plugin on the [Apache Cordova issue tracker](https://issues.apache.org/jira/issues/?jql=project%20%3D%20CB%20AND%20status%20in%20%28Open%2C%20%22In%20Progress%22%2C%20Reopened%29%20AND%20resolution%20%3D%20Unresolved%20AND%20component%20%3D%20%22Plugin%20InAppBrowser%22%20ORDER%20BY%20priority%20DESC%2C%20summary%20ASC%2C%20updatedDate%20DESC)
+:warning: Report issues on the [Apache Cordova issue tracker](https://issues.apache.org/jira/issues/?jql=project%20%3D%20CB%20AND%20status%20in%20%28Open%2C%20%22In%20Progress%22%2C%20Reopened%29%20AND%20resolution%20%3D%20Unresolved%20AND%20component%20%3D%20%22Plugin%20InAppBrowser%22%20ORDER%20BY%20priority%20DESC%2C%20summary%20ASC%2C%20updatedDate%20DESC)
 
 
 
@@ -102,7 +100,6 @@ instance, or the system browser.
     - __clearsessioncache__: set to `yes` to have the session cookie cache cleared before the new window is opened
     - __zoom__: set to `yes` to show Android browser's zoom controls, set to `no` to hide them.  Default value is `yes`.
     - __hardwareback__: set to `yes` to use the hardware back button to navigate backwards through the `InAppBrowser`'s history. If there is no previous page, the `InAppBrowser` will close.  The default value is `yes`, so you must set it to `no` if you want the back button to simply close the InAppBrowser.
-    - __mediaPlaybackRequiresUserAction__: Set to `yes` to prevent HTML5 audio or video from autoplaying (defaults to `no`).
 
     iOS only:
 
@@ -113,7 +110,7 @@ instance, or the system browser.
     - __clearsessioncache__: set to `yes` to have the session cookie cache cleared before the new window is opened
     - __toolbar__:  set to `yes` or `no` to turn the toolbar on or off for the InAppBrowser (defaults to `yes`)
     - __enableViewportScale__:  Set to `yes` or `no` to prevent viewport scaling through a meta tag (defaults to `no`).
-    - __mediaPlaybackRequiresUserAction__: Set to `yes` to prevent HTML5 audio or video from autoplaying (defaults to `no`).
+    - __mediaPlaybackRequiresUserAction__: Set to `yes` or `no` to prevent HTML5 audio or video from autoplaying (defaults to `no`).
     - __allowInlineMediaPlayback__: Set to `yes` or `no` to allow in-line HTML5 media playback, displaying within the browser window rather than a device-specific playback interface. The HTML's `video` element must also include the `webkit-playsinline` attribute (defaults to `no`)
     - __keyboardDisplayRequiresUserAction__: Set to `yes` or `no` to open the keyboard when form elements receive focus via JavaScript's `focus()` call (defaults to `yes`).
     - __suppressesIncrementalRendering__: Set to `yes` or `no` to wait until all new view content is received before being rendered (defaults to `no`).
@@ -200,7 +197,7 @@ The object returned from a call to `cordova.InAppBrowser.open`.
 - executeScript
 - insertCSS
 
-## InAppBrowser.addEventListener
+## addEventListener
 
 > Adds a listener for an event from the `InAppBrowser`.
 
@@ -246,7 +243,7 @@ The object returned from a call to `cordova.InAppBrowser.open`.
     var ref = cordova.InAppBrowser.open('http://apache.org', '_blank', 'location=yes');
     ref.addEventListener('loadstart', function(event) { alert(event.url); });
 
-## InAppBrowser.removeEventListener
+## removeEventListener
 
 > Removes a listener for an event from the `InAppBrowser`.
 
@@ -280,7 +277,7 @@ The function is passed an `InAppBrowserEvent` object.
     ref.addEventListener('loadstart', myCallback);
     ref.removeEventListener('loadstart', myCallback);
 
-## InAppBrowser.close
+## close
 
 > Closes the `InAppBrowser` window.
 
@@ -303,7 +300,7 @@ The function is passed an `InAppBrowserEvent` object.
     var ref = cordova.InAppBrowser.open('http://apache.org', '_blank', 'location=yes');
     ref.close();
 
-## InAppBrowser.show
+## show
 
 > Displays an InAppBrowser window that was opened hidden. Calling this has no effect if the InAppBrowser was already visible.
 
@@ -325,7 +322,7 @@ The function is passed an `InAppBrowserEvent` object.
     // some time later...
     ref.show();
 
-## InAppBrowser.executeScript
+## executeScript
 
 > Injects JavaScript code into the `InAppBrowser` window
 
@@ -367,7 +364,7 @@ The function is passed an `InAppBrowserEvent` object.
 
 Due to [MSDN docs](https://msdn.microsoft.com/en-us/library/windows.ui.xaml.controls.webview.invokescriptasync.aspx) the invoked script can return only string values, otherwise the parameter, passed to __callback__ will be `[null]`.
 
-## InAppBrowser.insertCSS
+## insertCSS
 
 > Injects CSS into the `InAppBrowser` window.
 

RELEASENOTES.md

@@ -20,34 +20,6 @@
 -->
 # Release Notes
 
-### 1.4.0 (Apr 15, 2016)
-* CB-7679 add fix for **iOS** upload.
-* CB-10944 `NoSuchMethodError` in `InAppBrowser` plugin
-* CB-10937 fix stretched icons
-* CB-10760 Fixing README for display on Cordova website
-* CB-10636 Add `JSHint` for plugins
-
-### 1.3.0 (Feb 09, 2016)
-* [CB-3360](https://issues.apache.org/jira/browse/CB-3360) Set custom inappbrowser user agent for android
-* [CB-10538](https://issues.apache.org/jira/browse/CB-10538) cordova-plugin-inappbrowser timeout issue
-* [CB-10395](https://issues.apache.org/jira/browse/CB-10395) InAppBrowser's WebView not storing cookies reliable on Android
-* Edit package.json license to match SPDX id
-* [CB-10305](https://issues.apache.org/jira/browse/CB-10305) Gray bar appears in the wrong place on iOS
-* [CB-7786](https://issues.apache.org/jira/browse/CB-7786) Support mediaPlaybackRequiresUserAction on Android
-* [CB-7500](https://issues.apache.org/jira/browse/CB-7500) executeScript with callback kills/blurs inAppBrowser window on Android
-
-### 1.2.1 (Feb 02, 2016)
-* [CB-10407](https://issues.apache.org/jira/browse/CB-10407) InAppBrowser not firing loadstart event on android
-* [CB-10428](https://issues.apache.org/jira/browse/CB-10428) Fix syntax error when browserifying inAppBrowser plugin
-* handle app store urls in system browser
-* [CB-6702](https://issues.apache.org/jira/browse/CB-6702) InAppBrowser hangs when opening more than one instance
-* [CB-10456](https://issues.apache.org/jira/browse/CB-10456) InAppBrowser is not closed if I close it programmatically on Android
-* [CB-10451](https://issues.apache.org/jira/browse/CB-10451) InAppBrowser: loadstart event is not triggered on Windows
-* [CB-10452](https://issues.apache.org/jira/browse/CB-10452) InAppBrowser: 'exit' event is not triggered on Windows
-* [CB-10454](https://issues.apache.org/jira/browse/CB-10454) InAppBrowser: 'loaderror' event does not have code and message on Windows
-* [CB-10450](https://issues.apache.org/jira/browse/CB-10450) InAppBrowser: Unable to get property 'canGoBack' of undefined on Windows
-* [CB-10441](https://issues.apache.org/jira/browse/CB-10441) Add auto tests for InAppBrowser plugin
-
 ### 1.2.0 (Jan 15, 2016)
 * CB-8180: Changing methods of interception in `WebViewClient` class
 * CB-10009 Improve `InAppBrowser` toolbar look and feel on **Windows**

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cordova-plugin-inappbrowser",
-  "version": "1.4.0",
+  "version": "1.2.1-dev",
   "description": "Cordova InAppBrowser Plugin",
   "cordova": {
     "id": "cordova-plugin-inappbrowser",
@@ -37,10 +37,6 @@
     "cordova-windows",
     "cordova-firefoxos"
   ],
-  "scripts": {
-    "test": "npm run jshint",
-    "jshint": "node node_modules/jshint/bin/jshint www && node node_modules/jshint/bin/jshint src && node node_modules/jshint/bin/jshint tests"
-  },
   "engines": [
     {
       "name": "cordova",
@@ -48,8 +44,5 @@
     }
   ],
   "author": "Apache Software Foundation",
-  "license": "Apache-2.0",
+  "license": "Apache 2.0"
-  "devDependencies": {
-    "jshint": "^2.6.0"
-  }
 }

plugin.xml

@@ -20,7 +20,7 @@
 
 <plugin xmlns="http://apache.org/cordova/ns/plugins/1.0"
            id="cordova-plugin-inappbrowser"
-      version="1.4.0">
+      version="1.2.1-dev">
 
     <name>InAppBrowser</name>
     <description>Cordova InAppBrowser Plugin</description>

src/android/InAppBrowser.java

@@ -19,6 +19,7 @@
 package org.apache.cordova.inappbrowser;
 
 import android.annotation.SuppressLint;
+import org.apache.cordova.inappbrowser.InAppBrowserDialog;
 import android.content.Context;
 import android.content.Intent;
 import android.provider.Browser;
@@ -40,14 +41,12 @@ import android.view.WindowManager.LayoutParams;
 import android.view.inputmethod.EditorInfo;
 import android.view.inputmethod.InputMethodManager;
 import android.webkit.CookieManager;
-import android.webkit.CookieSyncManager;
 import android.webkit.HttpAuthHandler;
 import android.webkit.WebSettings;
 import android.webkit.WebView;
 import android.webkit.WebViewClient;
+import android.widget.Button;
 import android.widget.EditText;
-import android.widget.ImageButton;
-import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.RelativeLayout;
 
@@ -86,7 +85,6 @@ public class InAppBrowser extends CordovaPlugin {
     private static final String CLEAR_ALL_CACHE = "clearcache";
     private static final String CLEAR_SESSION_CACHE = "clearsessioncache";
     private static final String HARDWARE_BACK_BUTTON = "hardwareback";
-    private static final String MEDIA_PLAYBACK_REQUIRES_USER_ACTION = "mediaPlaybackRequiresUserAction";
 
     private InAppBrowserDialog dialog;
     private WebView inAppWebView;
@@ -98,7 +96,6 @@ public class InAppBrowser extends CordovaPlugin {
     private boolean clearAllCache = false;
     private boolean clearSessionCache = false;
     private boolean hadwareBackButton = true;
-    private boolean mediaPlaybackRequiresUserGesture = false;
 
     /**
      * Executes the request and returns PluginResult.
@@ -203,7 +200,7 @@ public class InAppBrowser extends CordovaPlugin {
         else if (action.equals("injectScriptCode")) {
             String jsWrapper = null;
             if (args.getBoolean(1)) {
-                jsWrapper = String.format("(function(){prompt(JSON.stringify([eval(%%s)]), 'gap-iab://%s')})()", callbackContext.getCallbackId());
+                jsWrapper = String.format("prompt(JSON.stringify([eval(%%s)]), 'gap-iab://%s')", callbackContext.getCallbackId());
             }
             injectDeferredObject(args.getString(0), jsWrapper);
         }
@@ -365,22 +362,20 @@ public class InAppBrowser extends CordovaPlugin {
      * Closes the dialog
      */
     public void closeDialog() {
+        final WebView childView = this.inAppWebView;
+        // The JS protects against multiple calls, so this should happen only when
+        // closeDialog() is called by other native code.
+        if (childView == null) {
+            return;
+        }
         this.cordova.getActivity().runOnUiThread(new Runnable() {
             @Override
             public void run() {
-                final WebView childView = inAppWebView;
-                // The JS protects against multiple calls, so this should happen only when
-                // closeDialog() is called by other native code.
-                if (childView == null) {
-                    return;
-                }
-
                 childView.setWebViewClient(new WebViewClient() {
                     // NB: wait for about:blank before dismissing
                     public void onPageFinished(WebView view, String url) {
                         if (dialog != null) {
                             dialog.dismiss();
-                            dialog = null;
                         }
                     }
                 });
@@ -388,16 +383,16 @@ public class InAppBrowser extends CordovaPlugin {
                 // other than your app's UI thread, it can cause unexpected results."
                 // http://developer.android.com/guide/webapps/migrating.html#Threads
                 childView.loadUrl("about:blank");
-
-                try {
-                    JSONObject obj = new JSONObject();
-                    obj.put("type", EXIT_EVENT);
-                    sendUpdate(obj, false);
-                } catch (JSONException ex) {
-                    Log.d(LOG_TAG, "Should never happen");
-                }
             }
         });
+
+        try {
+            JSONObject obj = new JSONObject();
+            obj.put("type", EXIT_EVENT);
+            sendUpdate(obj, false);
+        } catch (JSONException ex) {
+            Log.d(LOG_TAG, "Should never happen");
+        }
     }
 
     /**
@@ -476,8 +471,6 @@ public class InAppBrowser extends CordovaPlugin {
         showLocationBar = true;
         showZoomControls = true;
         openWindowHidden = false;
-        mediaPlaybackRequiresUserGesture = false;
-
         if (features != null) {
             Boolean show = features.get(LOCATION);
             if (show != null) {
@@ -495,10 +488,6 @@ public class InAppBrowser extends CordovaPlugin {
             if (hardwareBack != null) {
                 hadwareBackButton = hardwareBack.booleanValue();
             }
-            Boolean mediaPlayback = features.get(MEDIA_PLAYBACK_REQUIRES_USER_ACTION);
-            if (mediaPlayback != null) {
-                mediaPlaybackRequiresUserGesture = mediaPlayback.booleanValue();
-            }
             Boolean cache = features.get(CLEAR_ALL_CACHE);
             if (cache != null) {
                 clearAllCache = cache.booleanValue();
@@ -530,12 +519,6 @@ public class InAppBrowser extends CordovaPlugin {
 
             @SuppressLint("NewApi")
             public void run() {
-
-                // CB-6702 InAppBrowser hangs when opening more than one instance
-                if (dialog != null) {
-                    dialog.dismiss();
-                };
-
                 // Let's create the main dialog
                 dialog = new InAppBrowserDialog(cordova.getActivity(), android.R.style.Theme_NoTitleBar);
                 dialog.getWindow().getAttributes().windowAnimations = android.R.style.Animation_Dialog;
@@ -564,7 +547,7 @@ public class InAppBrowser extends CordovaPlugin {
                 actionButtonContainer.setId(Integer.valueOf(1));
 
                 // Back button
-                ImageButton back = new ImageButton(cordova.getActivity());
+                Button back = new Button(cordova.getActivity());
                 RelativeLayout.LayoutParams backLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.MATCH_PARENT);
                 backLayoutParams.addRule(RelativeLayout.ALIGN_LEFT);
                 back.setLayoutParams(backLayoutParams);
@@ -573,12 +556,14 @@ public class InAppBrowser extends CordovaPlugin {
                 Resources activityRes = cordova.getActivity().getResources();
                 int backResId = activityRes.getIdentifier("ic_action_previous_item", "drawable", cordova.getActivity().getPackageName());
                 Drawable backIcon = activityRes.getDrawable(backResId);
-                back.setBackground(null);
+                if(android.os.Build.VERSION.SDK_INT < android.os.Build.VERSION_CODES.JELLY_BEAN)
-                back.setImageDrawable(backIcon);
+                {
-                back.setScaleType(ImageView.ScaleType.FIT_CENTER);
+                    back.setBackgroundDrawable(backIcon);
-                back.setPadding(0, this.dpToPixels(10), 0, this.dpToPixels(10));
+                }
-                back.getAdjustViewBounds();
+                else
-
+                {
+                    back.setBackground(backIcon);
+                }
                 back.setOnClickListener(new View.OnClickListener() {
                     public void onClick(View v) {
                         goBack();
@@ -586,7 +571,7 @@ public class InAppBrowser extends CordovaPlugin {
                 });
 
                 // Forward button
-                ImageButton forward = new ImageButton(cordova.getActivity());
+                Button forward = new Button(cordova.getActivity());
                 RelativeLayout.LayoutParams forwardLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.MATCH_PARENT);
                 forwardLayoutParams.addRule(RelativeLayout.RIGHT_OF, 2);
                 forward.setLayoutParams(forwardLayoutParams);
@@ -594,12 +579,14 @@ public class InAppBrowser extends CordovaPlugin {
                 forward.setId(Integer.valueOf(3));
                 int fwdResId = activityRes.getIdentifier("ic_action_next_item", "drawable", cordova.getActivity().getPackageName());
                 Drawable fwdIcon = activityRes.getDrawable(fwdResId);
-                forward.setBackground(null);
+                if(android.os.Build.VERSION.SDK_INT < android.os.Build.VERSION_CODES.JELLY_BEAN)
-                forward.setImageDrawable(fwdIcon);
+                {
-                forward.setScaleType(ImageView.ScaleType.FIT_CENTER);
+                    forward.setBackgroundDrawable(fwdIcon);
-                forward.setPadding(0, this.dpToPixels(10), 0, this.dpToPixels(10));
+                }
-                forward.getAdjustViewBounds();
+                else
-
+                {
+                    forward.setBackground(fwdIcon);
+                }
                 forward.setOnClickListener(new View.OnClickListener() {
                     public void onClick(View v) {
                         goForward();
@@ -630,7 +617,7 @@ public class InAppBrowser extends CordovaPlugin {
                 });
 
                 // Close/Done button
-                ImageButton close = new ImageButton(cordova.getActivity());
+                Button close = new Button(cordova.getActivity());
                 RelativeLayout.LayoutParams closeLayoutParams = new RelativeLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.MATCH_PARENT);
                 closeLayoutParams.addRule(RelativeLayout.ALIGN_PARENT_RIGHT);
                 close.setLayoutParams(closeLayoutParams);
@@ -638,12 +625,14 @@ public class InAppBrowser extends CordovaPlugin {
                 close.setId(Integer.valueOf(5));
                 int closeResId = activityRes.getIdentifier("ic_action_remove", "drawable", cordova.getActivity().getPackageName());
                 Drawable closeIcon = activityRes.getDrawable(closeResId);
-                close.setBackground(null);
+                if(android.os.Build.VERSION.SDK_INT < android.os.Build.VERSION_CODES.JELLY_BEAN)
-                close.setImageDrawable(closeIcon);
+                {
-                close.setScaleType(ImageView.ScaleType.FIT_CENTER);
+                    close.setBackgroundDrawable(closeIcon);
-                back.setPadding(0, this.dpToPixels(10), 0, this.dpToPixels(10));
+                }
-                close.getAdjustViewBounds();
+                else
-
+                {
+                    close.setBackground(closeIcon);
+                }
                 close.setOnClickListener(new View.OnClickListener() {
                     public void onClick(View v) {
                         closeDialog();
@@ -663,20 +652,6 @@ public class InAppBrowser extends CordovaPlugin {
                 settings.setBuiltInZoomControls(showZoomControls);
                 settings.setPluginState(android.webkit.WebSettings.PluginState.ON);
 
-                if(android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.JELLY_BEAN_MR1) {
-                    settings.setMediaPlaybackRequiresUserGesture(mediaPlaybackRequiresUserGesture);
-                }
-
-                String overrideUserAgent = preferences.getString("OverrideUserAgent", null);
-                String appendUserAgent = preferences.getString("AppendUserAgent", null);
-
-                if (overrideUserAgent != null) {
-                    settings.setUserAgentString(overrideUserAgent);
-                }
-                if (appendUserAgent != null) {
-                    settings.setUserAgentString(settings.getUserAgentString() + appendUserAgent);
-                }
-
                 //Toggle whether this is enabled or not!
                 Bundle appSettings = cordova.getActivity().getIntent().getExtras();
                 boolean enableDatabase = appSettings == null ? true : appSettings.getBoolean("InAppBrowserStorageEnabled", true);
@@ -887,13 +862,6 @@ public class InAppBrowser extends CordovaPlugin {
         public void onPageFinished(WebView view, String url) {
             super.onPageFinished(view, url);
 
-            // CB-10395 InAppBrowser's WebView not storing cookies reliable to local device storage
-            if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.LOLLIPOP) {
-                CookieManager.getInstance().flush();
-            } else {
-                CookieSyncManager.getInstance().sync();
-            }
-
             try {
                 JSONObject obj = new JSONObject();
                 obj.put("type", LOAD_STOP_EVENT);
@@ -955,3 +923,4 @@ public class InAppBrowser extends CordovaPlugin {
         }
     }
 }
+

src/browser/InAppBrowserProxy.js

@@ -19,7 +19,10 @@
  *
 */
 
-var modulemapper = require('cordova/modulemapper');
+var cordova = require('cordova'),
+    channel = require('cordova/channel'),
+    modulemapper = require('cordova/modulemapper'),
+    urlutil = require('cordova/urlutil');
 
 var browserWrap,
     popup,
@@ -66,7 +69,8 @@ var IAB = {
     open: function (win, lose, args) {
         var strUrl = args[0],
             target = args[1],
-            features = args[2];
+            features = args[2],
+            url;
 
         if (target === "_self" || !target) {
             window.location = strUrl;
@@ -189,9 +193,7 @@ var IAB = {
         if (browserWrap && popup) {
             try {
                 popup.contentWindow.eval(code);
-                if (hasCallback) {
+                hasCallback && win([]);
-                    win([]);
-                }
             } catch(e) {
                 console.error('Error occured while trying to injectScriptCode: ' + JSON.stringify(e));
             }
@@ -201,25 +203,19 @@ var IAB = {
     injectScriptFile: function (win, fail, args) {
         var msg = 'Browser cordova-plugin-inappbrowser injectScriptFile is not yet implemented';
         console.warn(msg);
-        if (fail) {
+        fail && fail(msg);
-            fail(msg);
-        }
     }, 
 
     injectStyleCode: function (win, fail, args) {
         var msg = 'Browser cordova-plugin-inappbrowser injectStyleCode is not yet implemented';
         console.warn(msg);
-        if (fail) {
+        fail && fail(msg);
-            fail(msg);
-        }
     },
 
     injectStyleFile: function (win, fail, args) {
         var msg = 'Browser cordova-plugin-inappbrowser injectStyleFile is not yet implemented';
         console.warn(msg);
-        if (fail) {
+        fail && fail(msg);
-            fail(msg);
-        }
     }
 };
 

src/firefoxos/InAppBrowserProxy.js

@@ -21,7 +21,9 @@
 
 // https://developer.mozilla.org/en-US/docs/WebAPI/Browser
 
-var modulemapper = require('cordova/modulemapper');
+var cordova = require('cordova'),
+    channel = require('cordova/channel'),
+    modulemapper = require('cordova/modulemapper');
 
 var origOpenFunc = modulemapper.getOriginalSymbol(window, 'window.open');
 var browserWrap;
@@ -47,7 +49,9 @@ var IABExecs = {
         var strUrl = args[0],
             target = args[1],
             features_string = args[2] || "location=yes", //location=yes is default
-            features = {};
+            features = {},
+            url,
+            elem;
 
         var features_list = features_string.split(',');
         features_list.forEach(function(feature) {
@@ -111,7 +115,7 @@ var IABExecs = {
             back.classList.add('inAppBrowserBack');
             forward.classList.add('inAppBrowserForward');
 
-            var checkForwardBackward = function () {
+            function checkForwardBackward() {
                 var backReq = browserElem.getCanGoBack();
                 backReq.onsuccess = function() {
                     if (this.result) {
@@ -119,7 +123,7 @@ var IABExecs = {
                     } else {
                         back.classList.add('disabled');
                     }
-                };
+                }
                 var forwardReq = browserElem.getCanGoForward();
                 forwardReq.onsuccess = function() {
                     if (this.result) {
@@ -127,7 +131,7 @@ var IABExecs = {
                     } else {
                         forward.classList.add('disabled');
                     }
-                };
+                }
             };
 
             browserElem.addEventListener('mozbrowserloadend', checkForwardBackward);
@@ -159,16 +163,16 @@ var IABExecs = {
                 win({
                     type:'loadstart',
                     url : e.detail
-                });
+                })
             }, false);
             browserElem.addEventListener('mozbrowserloadend', function(e){
-                win({type:'loadstop'});
+                win({type:'loadstop'})
             }, false);
             browserElem.addEventListener('mozbrowsererror', function(e){
-                win({type:'loaderror'});
+                win({type:'loaderror'})
             }, false);
             browserElem.addEventListener('mozbrowserclose', function(e){
-                win({type:'exit'});
+                win({type:'exit'})
             }, false);
         } else {
             window.location = strUrl;

src/ios/CDVInAppBrowser.m

@@ -390,14 +390,7 @@
             [self.commandDelegate sendPluginResult:pluginResult callbackId:scriptCallbackId];
             return NO;
         }
-    } 
+    } else if ((self.callbackId != nil) && isTopLevelNavigation) {
-    //if is an app store link, let the system handle it, otherwise it fails to load it
-    else if ([[ url scheme] isEqualToString:@"itms-appss"] || [[ url scheme] isEqualToString:@"itms-apps"]) {
-        [theWebView stopLoading];
-        [self openInSystem:url];
-        return NO;
-    }
-    else if ((self.callbackId != nil) && isTopLevelNavigation) {
         // Send a loadstart event for each top-level navigation (includes redirects).
         CDVPluginResult* pluginResult = [CDVPluginResult resultWithStatus:CDVCommandStatus_OK
                                                       messageAsDictionary:@{@"type":@"loadstart", @"url":[url absoluteString]}];
@@ -755,10 +748,6 @@
     return UIStatusBarStyleDefault;
 }
 
-- (BOOL)prefersStatusBarHidden {
-    return NO;
-}
-
 - (void)close
 {
     [CDVUserAgentUtil releaseLock:&_userAgentLockToken];
@@ -998,38 +987,19 @@
 
 @implementation CDVInAppBrowserNavigationController : UINavigationController
 
-- (void) dismissViewControllerAnimated:(BOOL)flag completion:(void (^)(void))completion {
-    if ( self.presentedViewController) {
-        [super dismissViewControllerAnimated:flag completion:completion];
-    }
-}
-
 - (void) viewDidLoad {
 
     CGRect frame = [UIApplication sharedApplication].statusBarFrame;
 
     // simplified from: http://stackoverflow.com/a/25669695/219684
 
-    UIToolbar* bgToolbar = [[UIToolbar alloc] initWithFrame:[self invertFrameIfNeeded:frame]];
+    UIToolbar* bgToolbar = [[UIToolbar alloc] initWithFrame:frame];
     bgToolbar.barStyle = UIBarStyleDefault;
-    [bgToolbar setAutoresizingMask:UIViewAutoresizingFlexibleWidth];
     [self.view addSubview:bgToolbar];
 
     [super viewDidLoad];
 }
 
-- (CGRect) invertFrameIfNeeded:(CGRect)rect {
-    // We need to invert since on iOS 7 frames are always in Portrait context
-    if (!IsAtLeastiOSVersion(@"8.0")) {
-        if (UIInterfaceOrientationIsLandscape([[UIApplication sharedApplication] statusBarOrientation])) {
-            CGFloat temp = rect.size.width;
-            rect.size.width = rect.size.height;
-            rect.size.height = temp;
-        }
-        rect.origin = CGPointZero;
-    }
-    return rect;
-}
 
 #pragma mark CDVScreenOrientationDelegate
 

src/ubuntu/InAppBrowser_escapeScript.js

@@ -19,9 +19,6 @@
  *
 */
 
-/* jshint -W061 */
-/* global oxide */
-
 oxide.addMessageHandler("EXECUTE", function(msg) {
     var code = msg.args.code;
     try {

src/windows/InAppBrowserProxy.js

@@ -19,10 +19,13 @@
  *
 */
 
-/* jslint sloppy:true */
+/*jslint sloppy:true */
-/* global Windows:true, setImmediate */
+/*global Windows:true, require, document, setTimeout, window, module */
+
+
 
 var cordova = require('cordova'),
+    channel = require('cordova/channel'),
     urlutil = require('cordova/urlutil');
 
 var browserWrap,
@@ -32,12 +35,11 @@ var browserWrap,
     backButton,
     forwardButton,
     closeButton,
-    bodyOverflowStyle,
+    bodyOverflowStyle;
-    navigationEventsCallback;
 
 // x-ms-webview is available starting from Windows 8.1 (platformId is 'windows')
 // http://msdn.microsoft.com/en-us/library/windows/apps/dn301831.aspx
-var isWebViewAvailable = cordova.platformId === 'windows';
+var isWebViewAvailable = cordova.platformId == 'windows';
 
 function attachNavigationEvents(element, callback) {
     if (isWebViewAvailable) {
@@ -46,21 +48,17 @@ function attachNavigationEvents(element, callback) {
         });
 
         element.addEventListener("MSWebViewNavigationCompleted", function (e) {
-            if (e.isSuccess) {
+            callback({ type: e.isSuccess ? "loadstop" : "loaderror", url: e.uri}, {keepCallback: true});
-                callback({ type: "loadstop", url: e.uri }, { keepCallback: true });
-            } else {
-                callback({ type: "loaderror", url: e.uri, code: e.webErrorStatus, message: "Navigation failed with error code " + e.webErrorStatus}, { keepCallback: true });
-            }
         });
 
         element.addEventListener("MSWebViewUnviewableContentIdentified", function (e) {
             // WebView found the content to be not HTML.
             // http://msdn.microsoft.com/en-us/library/windows/apps/dn609716.aspx
-            callback({ type: "loaderror", url: e.uri, code: e.webErrorStatus, message: "Navigation failed with error code " + e.webErrorStatus}, { keepCallback: true });
+            callback({ type: "loaderror", url: e.uri}, {keepCallback: true});
         });
 
         element.addEventListener("MSWebViewContentLoading", function (e) {
-            if (navigationButtonsDiv && popup) {
+            if (navigationButtonsDiv) {
                 if (popup.canGoBack) {
                     backButton.removeAttribute("disabled");
                 } else {
@@ -94,232 +92,209 @@ function attachNavigationEvents(element, callback) {
 
 var IAB = {
     close: function (win, lose) {
-        setImmediate(function () {
+        if (browserWrap) {
-            if (browserWrap) {
+            if (win) win({ type: "exit" });
-                if (navigationEventsCallback) {
-                    navigationEventsCallback({ type: "exit" });
-                }
 
-                browserWrap.parentNode.removeChild(browserWrap);
+            browserWrap.parentNode.removeChild(browserWrap);
-                // Reset body overflow style to initial value
+            // Reset body overflow style to initial value
-                document.body.style.msOverflowStyle = bodyOverflowStyle;
+            document.body.style.msOverflowStyle = bodyOverflowStyle;
-                browserWrap = null;
+            browserWrap = null;
-                popup = null;
+            popup = null;
-            }
+        }
-        });
     },
     show: function (win, lose) {
-        setImmediate(function () {
+        if (browserWrap) {
-            if (browserWrap) {
+            browserWrap.style.display = "block";
-                browserWrap.style.display = "block";
+        }
-            }
-        });
     },
     open: function (win, lose, args) {
-        // make function async so that we can add navigation events handlers before view is loaded and navigation occured
+        var strUrl = args[0],
-        setImmediate(function () {
+            target = args[1],
-            var strUrl = args[0],
+            features = args[2],
-                target = args[1],
+            url;
-                features = args[2],
-                url;
 
-            navigationEventsCallback = win;
+        if (target === "_system") {
+            url = new Windows.Foundation.Uri(strUrl);
+            Windows.System.Launcher.launchUriAsync(url);
+        } else if (target === "_self" || !target) {
+            window.location = strUrl;
+        } else {
+            // "_blank" or anything else
+            if (!browserWrap) {
+                var browserWrapStyle = document.createElement('link');
+                browserWrapStyle.rel = "stylesheet";
+                browserWrapStyle.type = "text/css";
+                browserWrapStyle.href = urlutil.makeAbsolute("/www/css/inappbrowser.css");
 
-            if (target === "_system") {
+                document.head.appendChild(browserWrapStyle);
-                url = new Windows.Foundation.Uri(strUrl);
-                Windows.System.Launcher.launchUriAsync(url);
-            } else if (target === "_self" || !target) {
-                window.location = strUrl;
-            } else {
-                // "_blank" or anything else
-                if (!browserWrap) {
-                    var browserWrapStyle = document.createElement('link');
-                    browserWrapStyle.rel = "stylesheet";
-                    browserWrapStyle.type = "text/css";
-                    browserWrapStyle.href = urlutil.makeAbsolute("/www/css/inappbrowser.css");
 
-                    document.head.appendChild(browserWrapStyle);
+                browserWrap = document.createElement("div");
+                browserWrap.className = "inAppBrowserWrap";
 
-                    browserWrap = document.createElement("div");
+                if (features.indexOf("fullscreen=yes") > -1) {
-                    browserWrap.className = "inAppBrowserWrap";
+                    browserWrap.classList.add("inAppBrowserWrapFullscreen");
-
-                    if (features.indexOf("fullscreen=yes") > -1) {
-                        browserWrap.classList.add("inAppBrowserWrapFullscreen");
-                    }
-
-                    // Save body overflow style to be able to reset it back later
-                    bodyOverflowStyle = document.body.style.msOverflowStyle;
-
-                    browserWrap.onclick = function () {
-                        setTimeout(function () {
-                            IAB.close(navigationEventsCallback);
-                        }, 0);
-                    };
-
-                    document.body.appendChild(browserWrap);
-                    // Hide scrollbars for the whole body while inappbrowser's window is open
-                    document.body.style.msOverflowStyle = "none";
                 }
 
-                if (features.indexOf("hidden=yes") !== -1) {
+                // Save body overflow style to be able to reset it back later
-                    browserWrap.style.display = "none";
+                bodyOverflowStyle = document.body.style.msOverflowStyle;
-                }
 
-                popup = document.createElement(isWebViewAvailable ? "x-ms-webview" : "iframe");
+                browserWrap.onclick = function () {
-                if (popup instanceof HTMLIFrameElement) {
+                    setTimeout(function () {
-                    // For iframe we need to override bacground color of parent element here
+                        IAB.close(win);
-                    // otherwise pages without background color set will have transparent background
+                    }, 0);
-                    popup.style.backgroundColor = "white";
+                };
-                }
-                popup.style.borderWidth = "0px";
-                popup.style.width = "100%";
 
-                browserWrap.appendChild(popup);
+                document.body.appendChild(browserWrap);
-
+                // Hide scrollbars for the whole body while inappbrowser's window is open
-                if (features.indexOf("location=yes") !== -1 || features.indexOf("location") === -1) {
+                document.body.style.msOverflowStyle = "none";
-                    popup.style.height = "calc(100% - 70px)";
-
-                    navigationButtonsDiv = document.createElement("div");
-                    navigationButtonsDiv.className = "inappbrowser-app-bar";
-                    navigationButtonsDiv.onclick = function (e) {
-                        e.cancelBubble = true;
-                    };
-
-                    navigationButtonsDivInner = document.createElement("div");
-                    navigationButtonsDivInner.className = "inappbrowser-app-bar-inner";
-                    navigationButtonsDivInner.onclick = function (e) {
-                        e.cancelBubble = true;
-                    };
-
-                    backButton = document.createElement("div");
-                    backButton.innerText = "back";
-                    backButton.className = "app-bar-action action-back";
-                    backButton.addEventListener("click", function (e) {
-                        if (popup.canGoBack)
-                            popup.goBack();
-                    });
-
-                    forwardButton = document.createElement("div");
-                    forwardButton.innerText = "forward";
-                    forwardButton.className = "app-bar-action action-forward";
-                    forwardButton.addEventListener("click", function (e) {
-                        if (popup.canGoForward)
-                            popup.goForward();
-                    });
-
-                    closeButton = document.createElement("div");
-                    closeButton.innerText = "close";
-                    closeButton.className = "app-bar-action action-close";
-                    closeButton.addEventListener("click", function (e) {
-                        setTimeout(function () {
-                            IAB.close(navigationEventsCallback);
-                        }, 0);
-                    });
-
-                    if (!isWebViewAvailable) {
-                        // iframe navigation is not yet supported
-                        backButton.setAttribute("disabled", "true");
-                        forwardButton.setAttribute("disabled", "true");
-                    }
-
-                    navigationButtonsDivInner.appendChild(backButton);
-                    navigationButtonsDivInner.appendChild(forwardButton);
-                    navigationButtonsDivInner.appendChild(closeButton);
-                    navigationButtonsDiv.appendChild(navigationButtonsDivInner);
-
-                    browserWrap.appendChild(navigationButtonsDiv);
-                } else {
-                    popup.style.height = "100%";
-                }
-
-                // start listening for navigation events
-                attachNavigationEvents(popup, navigationEventsCallback);
-
-                if (isWebViewAvailable) {
-                    strUrl = strUrl.replace("ms-appx://", "ms-appx-web://");
-                }
-                popup.src = strUrl;
             }
-        });
+
+            if (features.indexOf("hidden=yes") !== -1) {
+                browserWrap.style.display = "none";
+            }
+
+            popup = document.createElement(isWebViewAvailable ? "x-ms-webview" : "iframe");
+            if (popup instanceof HTMLIFrameElement) {
+                // For iframe we need to override bacground color of parent element here
+                // otherwise pages without background color set will have transparent background
+                popup.style.backgroundColor = "white";
+            }
+            popup.style.borderWidth = "0px";
+            popup.style.width = "100%";
+
+            browserWrap.appendChild(popup);
+
+            if (features.indexOf("location=yes") !== -1 || features.indexOf("location") === -1) {
+                popup.style.height = "calc(100% - 70px)";
+
+                navigationButtonsDiv = document.createElement("div");
+                navigationButtonsDiv.className = "inappbrowser-app-bar";
+                navigationButtonsDiv.onclick = function (e) {
+                    e.cancelBubble = true;
+                };
+
+                navigationButtonsDivInner = document.createElement("div");
+                navigationButtonsDivInner.className = "inappbrowser-app-bar-inner"
+                navigationButtonsDivInner.onclick = function (e) {
+                    e.cancelBubble = true;
+                };
+
+                backButton = document.createElement("div");
+                backButton.innerText = "back";
+                backButton.className = "app-bar-action action-back";
+                backButton.addEventListener("click", function (e) {
+                    if (popup.canGoBack)
+                        popup.goBack();
+                });
+
+                forwardButton = document.createElement("div");
+                forwardButton.innerText = "forward";
+                forwardButton.className = "app-bar-action action-forward";
+                forwardButton.addEventListener("click", function (e) {
+                    if (popup.canGoForward)
+                        popup.goForward();
+                });
+
+                closeButton = document.createElement("div");
+                closeButton.innerText = "close";
+                closeButton.className = "app-bar-action action-close";
+                closeButton.addEventListener("click", function (e) {
+                    setTimeout(function () {
+                        IAB.close(win);
+                    }, 0);
+                });
+
+                if (!isWebViewAvailable) {
+                    // iframe navigation is not yet supported
+                    backButton.setAttribute("disabled", "true");
+                    forwardButton.setAttribute("disabled", "true");
+                }
+
+                navigationButtonsDivInner.appendChild(backButton);
+                navigationButtonsDivInner.appendChild(forwardButton);
+                navigationButtonsDivInner.appendChild(closeButton);
+                navigationButtonsDiv.appendChild(navigationButtonsDivInner);
+
+                browserWrap.appendChild(navigationButtonsDiv);
+            } else {
+                popup.style.height = "100%";
+            }
+
+            // start listening for navigation events
+            attachNavigationEvents(popup, win);
+
+            if (isWebViewAvailable) {
+                strUrl = strUrl.replace("ms-appx://", "ms-appx-web://");
+            }
+            popup.src = strUrl;
+        }
     },
 
     injectScriptCode: function (win, fail, args) {
-        setImmediate(function () {
+        var code = args[0],
-            var code = args[0],
+            hasCallback = args[1];
-                hasCallback = args[1];
 
-            if (isWebViewAvailable && browserWrap && popup) {
+        if (isWebViewAvailable && browserWrap && popup) {
-                var op = popup.invokeScriptAsync("eval", code);
+            var op = popup.invokeScriptAsync("eval", code);
-                op.oncomplete = function (e) {
+            op.oncomplete = function (e) {
-                    if (hasCallback) {
+                // return null if event target is unavailable by some reason
-                        // return null if event target is unavailable by some reason
+                var result = (e && e.target) ? [e.target.result] : [null];
-                        var result = (e && e.target) ? [e.target.result] : [null];
+                hasCallback && win(result);
-                        win(result);
+            };
-                    }
+            op.onerror = function () { };
-                };
+            op.start();
-                op.onerror = function () { };
+        }
-                op.start();
-            }
-        });
     },
 
     injectScriptFile: function (win, fail, args) {
-        setImmediate(function () {
+        var filePath = args[0],
-            var filePath = args[0],
+            hasCallback = args[1];
-                hasCallback = args[1];
 
-            if (!!filePath) {
+        if (!!filePath) {
-                filePath = urlutil.makeAbsolute(filePath);
+            filePath = urlutil.makeAbsolute(filePath);
-            }
+        }
 
-            if (isWebViewAvailable && browserWrap && popup) {
+        if (isWebViewAvailable && browserWrap && popup) {
-                var uri = new Windows.Foundation.Uri(filePath);
+            var uri = new Windows.Foundation.Uri(filePath);
-                Windows.Storage.StorageFile.getFileFromApplicationUriAsync(uri).done(function (file) {
+            Windows.Storage.StorageFile.getFileFromApplicationUriAsync(uri).done(function (file) {
-                    Windows.Storage.FileIO.readTextAsync(file).done(function (code) {
+                Windows.Storage.FileIO.readTextAsync(file).done(function (code) {
-                        var op = popup.invokeScriptAsync("eval", code);
+                    var op = popup.invokeScriptAsync("eval", code);
-                        op.oncomplete = function(e) {
+                    op.oncomplete = function(e) {
-                            if (hasCallback) {
+                        var result = [e.target.result];
-                                var result = [e.target.result];
+                        hasCallback && win(result);
-                                win(result);
+                    };
-                            }
+                    op.onerror = function () { };
-                        };
+                    op.start();
-                        op.onerror = function () { };
-                        op.start();
-                    });
                 });
-            }
+            });
-        });
+        }
     },
 
     injectStyleCode: function (win, fail, args) {
-        setImmediate(function () {
+        var code = args[0],
-            var code = args[0],
+            hasCallback = args[1];
-                hasCallback = args[1];
 
-            if (isWebViewAvailable && browserWrap && popup) {
+        if (isWebViewAvailable && browserWrap && popup) {
-                injectCSS(popup, code, hasCallback && win);
+            injectCSS(popup, code, hasCallback && win);
-            }
+        }
-        });
     },
 
     injectStyleFile: function (win, fail, args) {
-        setImmediate(function () {
+        var filePath = args[0],
-            var filePath = args[0],
+            hasCallback = args[1];
-                hasCallback = args[1];
 
-            filePath = filePath && urlutil.makeAbsolute(filePath);
+        filePath = filePath && urlutil.makeAbsolute(filePath);
 
-            if (isWebViewAvailable && browserWrap && popup) {
+        if (isWebViewAvailable && browserWrap && popup) {
-                var uri = new Windows.Foundation.Uri(filePath);
+            var uri = new Windows.Foundation.Uri(filePath);
-                Windows.Storage.StorageFile.getFileFromApplicationUriAsync(uri).then(function (file) {
+            Windows.Storage.StorageFile.getFileFromApplicationUriAsync(uri).then(function (file) {
-                    return Windows.Storage.FileIO.readTextAsync(file);
+                return Windows.Storage.FileIO.readTextAsync(file);
-                }).done(function (code) {
+            }).done(function (code) {
-                    injectCSS(popup, code, hasCallback && win);
+                injectCSS(popup, code, hasCallback && win);
-                }, function () {
+            }, function () {
-                    // no-op, just catch an error
+                // no-op, just catch an error
-                });
+            });
-            }
+        }
-        });
     }
 };
 
@@ -331,9 +306,7 @@ function injectCSS (webView, cssCode, callback) {
 
     var op = webView.invokeScriptAsync("eval", evalWrapper);
     op.oncomplete = function() {
-        if (callback) {
+        callback && callback([]);
-            callback([]);
-        }
     };
     op.onerror = function () { };
     op.start();

tests/plugin.xml

@@ -20,12 +20,10 @@
 
 <plugin xmlns="http://apache.org/cordova/ns/plugins/1.0"
     id="cordova-plugin-inappbrowser-tests"
-    version="1.4.0">
+    version="1.2.1-dev">
     <name>Cordova InAppBrowser Plugin Tests</name>
     <license>Apache 2.0</license>
 
-    <dependency id="cordova-plugin-dialogs" />
-
     <js-module src="tests.js" name="tests">
     </js-module>
 

tests/resources/inject.js

@@ -16,5 +16,5 @@
  * specific language governing permissions and limitations
  * under the License.
 */
-var d = document.getElementById("header");
+var d = document.getElementById("header")
 d.innerHTML = "Script file successfully injected";

tests/resources/video.html

@@ -38,8 +38,5 @@
       <button onclick="document.getElementById('player').play()"> play </button>
       <button onclick="document.getElementById('player').pause()"> pause </button>
     </div>
-    <script>
-      document.getElementById('player').play();
-    </script>
   </body>
 </html>

tests/tests.js

@@ -19,9 +19,6 @@
  *
 */
 
-/* jshint jasmine: true */
-/* global MSApp */
-
 var cordova = require('cordova');
 var isWindows = cordova.platformId == 'windows';
 
@@ -37,64 +34,50 @@ exports.defineAutoTests = function () {
 
         it("inappbrowser.spec.2 should contain open function", function () {
             expect(cordova.InAppBrowser.open).toBeDefined();
-            expect(cordova.InAppBrowser.open).toEqual(jasmine.any(Function));
+            expect(typeof cordova.InAppBrowser.open === 'function').toBe(true);
         });
     });
 
     describe('open method', function () {
 
-        var iabInstance;
+        var iabInsance;
         var originalTimeout;
-        var url = 'https://dist.apache.org/repos/dist/dev/cordova/';
+        var url = 'http://apache.org/';
-        var badUrl = 'http://bad-uri/';
 
         beforeEach(function () {
             // increase timeout to ensure test url could be loaded within test time
             originalTimeout = jasmine.DEFAULT_TIMEOUT_INTERVAL;
-            jasmine.DEFAULT_TIMEOUT_INTERVAL = 30000;
+            jasmine.DEFAULT_TIMEOUT_INTERVAL = 10000;
 
-            iabInstance = null;
+            iabInsance = null;
         });
 
-        afterEach(function (done) {
+        afterEach(function () {
             // restore original timeout
             jasmine.DEFAULT_TIMEOUT_INTERVAL = originalTimeout;
 
-            if (iabInstance !== null && iabInstance.close) {
+            if (iabInsance && iabInsance.close) {
-                iabInstance.close();
+                iabInsance.close();
             }
-            iabInstance = null;
+            iabInsance = null;
-            // add some extra time so that iab dialog is closed
-            setTimeout(done, 2000);
         });
 
         function verifyEvent(evt, type) {
             expect(evt).toBeDefined();
             expect(evt.type).toEqual(type);
-            if (type !== 'exit') { // `exit` event does not have url field
+            expect(evt.url).toEqual(url);
-                expect(evt.url).toEqual(url);
-            }
         }
 
-        function verifyLoadErrorEvent(evt) {
+        it("inappbrowser.spec.3 should retun InAppBrowser class instance", function () {
-            expect(evt).toBeDefined();
+            iabInsance = cordova.InAppBrowser.open(url, '_blank');
-            expect(evt.type).toEqual('loaderror');
-            expect(evt.url).toEqual(badUrl);
-            expect(evt.code).toEqual(jasmine.any(Number));
-            expect(evt.message).toEqual(jasmine.any(String));
-        }
 
-        it("inappbrowser.spec.3 should retun InAppBrowser instance with required methods", function () {
+            expect(iabInsance).toBeDefined();
-            iabInstance = cordova.InAppBrowser.open(url, '_blank');
 
-            expect(iabInstance).toBeDefined();
+            expect(iabInsance.addEventListener).toBeDefined();
+            expect(typeof iabInsance.addEventListener === 'function').toBe(true);
+            expect(iabInsance.close).toBeDefined();
+            expect(typeof iabInsance.close === 'function').toBe(true);
 
-            expect(iabInstance.addEventListener).toEqual(jasmine.any(Function));
-            expect(iabInstance.removeEventListener).toEqual(jasmine.any(Function));
-            expect(iabInstance.close).toEqual(jasmine.any(Function));
-            expect(iabInstance.show).toEqual(jasmine.any(Function));
-            expect(iabInstance.executeScript).toEqual(jasmine.any(Function));
-            expect(iabInstance.insertCSS).toEqual(jasmine.any(Function));
         });
 
         it("inappbrowser.spec.4 should support loadstart and loadstop events", function (done) {
@@ -102,32 +85,14 @@ exports.defineAutoTests = function () {
                 verifyEvent(evt, 'loadstart');
             });
 
-            iabInstance = cordova.InAppBrowser.open(url, '_blank');
+            iabInsance = cordova.InAppBrowser.open(url, '_blank');
-            iabInstance.addEventListener('loadstart', onLoadStart);
+            iabInsance.addEventListener('loadstart', onLoadStart);
-            iabInstance.addEventListener('loadstop', function (evt) {
+            iabInsance.addEventListener('loadstop', function (evt) {
                 verifyEvent(evt, 'loadstop');
                 expect(onLoadStart).toHaveBeenCalled();
                 done();
             });
         });
-
-        it("inappbrowser.spec.5 should support exit event", function (done) {
-            iabInstance = cordova.InAppBrowser.open(url, '_blank');
-            iabInstance.addEventListener('exit', function (evt) {
-                verifyEvent(evt, 'exit');
-                done();
-            });
-            iabInstance.close();
-            iabInstance = null;
-        });
-
-        it("inappbrowser.spec.6 should support loaderror event", function (done) {
-            iabInstance = cordova.InAppBrowser.open(badUrl, '_blank');
-            iabInstance.addEventListener('loaderror', function (evt) {
-                verifyLoadErrorEvent(evt);
-                done();
-            });
-        });
     });
 };
 
@@ -186,13 +151,13 @@ exports.defineManualTests = function (contentEl, createActionButton) {
                 if (e.url != lastLoadStartURL) {
                     alert('Unexpected: ' + e.type + ' event.url != loadstart\'s event.url');
                 }
-                if (numExpectedRedirects === 0 && counts.loadstart !== 1) {
+                if (numExpectedRedirects === 0 && counts['loadstart'] !== 1) {
                     // Do allow a loaderror without a loadstart (e.g. in the case of an invalid URL).
-                    if (!(e.type == 'loaderror' && counts.loadstart === 0)) {
+                    if (!(e.type == 'loaderror' && counts['loadstart'] === 0)) {
-                        alert('Unexpected: got multiple loadstart events. (' + counts.loadstart + ')');
+                        alert('Unexpected: got multiple loadstart events. (' + counts['loadstart'] + ')');
                     }
-                } else if (numExpectedRedirects > 0 && counts.loadstart < (numExpectedRedirects + 1)) {
+                } else if (numExpectedRedirects > 0 && counts['loadstart'] < (numExpectedRedirects + 1)) {
-                    alert('Unexpected: should have got at least ' + (numExpectedRedirects + 1) + ' loadstart events, but got ' + counts.loadstart);
+                    alert('Unexpected: should have got at least ' + (numExpectedRedirects + 1) + ' loadstart events, but got ' + counts['loadstart']);
                 }
                 wasReset = true;
                 numExpectedRedirects = 0;
@@ -200,7 +165,7 @@ exports.defineManualTests = function (contentEl, createActionButton) {
             }
             // Verify that loadend / loaderror was called.
             if (e.type == 'exit') {
-                var numStopEvents = counts.loadstop + counts.loaderror;
+                var numStopEvents = counts['loadstop'] + counts['loaderror'];
                 if (numStopEvents === 0 && !wasReset) {
                     alert('Unexpected: browser closed without a loadstop or loaderror.');
                 } else if (numStopEvents > 1) {
@@ -426,11 +391,7 @@ exports.defineManualTests = function (contentEl, createActionButton) {
 
     var video_tag_tests = '<h1>Video tag</h1>' +
         '<div id="openRemoteVideo"></div>' +
-        'Expected result: open successfully in InAppBrowser with an embedded video plays automatically on iOS and Android.' +
+        'Expected result: open successfully in InAppBrowser with an embedded video that works after clicking the "play" button.';
-        '<div id="openRemoteNeedUserNoVideo"></div>' +
-        'Expected result: open successfully in InAppBrowser with an embedded video plays automatically on iOS and Android.' +
-        '<div id="openRemoteNeedUserYesVideo"></div>' +
-        'Expected result: open successfully in InAppBrowser with an embedded video does not play automatically on iOS and Android but rather works after clicking the "play" button.';
 
     var local_with_anchor_tag_tests = '<h1>Local with anchor tag</h1>' +
         '<div id="openAnchor1"></div>' +
@@ -618,12 +579,6 @@ exports.defineManualTests = function (contentEl, createActionButton) {
     createActionButton('Remote Video', function () {
         doOpen(videohtml, '_blank');
     }, 'openRemoteVideo');
-    createActionButton('Remote Need User No Video', function () {
-        doOpen(videohtml, '_blank', 'mediaPlaybackRequiresUserAction=no');
-    }, 'openRemoteNeedUserNoVideo');
-    createActionButton('Remote Need User Yes Video', function () {
-        doOpen(videohtml, '_blank', 'mediaPlaybackRequiresUserAction=yes');
-    }, 'openRemoteNeedUserYesVideo');
 
     //Local With Anchor Tag
     createActionButton('Anchor1', function () {
@@ -633,3 +588,4 @@ exports.defineManualTests = function (contentEl, createActionButton) {
         doOpen(localhtml + '#anchor2', '_blank');
     }, 'openAnchor2');
 };
+

www/windows8/InAppBrowserProxy.js

@@ -22,6 +22,10 @@
 /*jslint sloppy:true */
 /*global Windows:true, require, document, setTimeout, window, module */
 
+
+var cordova = require('cordova'),
+    channel = require('cordova/channel');
+
 var browserWrap;
 
 var IAB = {
@@ -41,6 +45,7 @@ var IAB = {
     open: function (win, lose, args) {
         var strUrl = args[0],
             target = args[1],
+            features = args[2],
             url,
             elem;