zhaidi/cordova-plugin-inappbrowser
zhaidi/cordova-plugin-inappbrowser
1
0
Fork 0
forked from public/cordova-plugin-inappbrowser
Code Pull Requests Activity

CB-5021 Expose closeDialog() as a public function and make it safe to call multiple times.

Browse Source
This commit is contained in:
Andrew Grieve 2013-10-10 08:07:46 -04:00
parent ef5eddac9a
commit 8cd786b603
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/android/InAppBrowser.java
View File

@ -157,7 +157,7 @@ public class InAppBrowser extends CordovaPlugin {
} }
else if (action.equals("close")) { else if (action.equals("close")) {
closeDialog(); closeDialog();
this.callbackContext.sendPluginResult(new PluginResult(PluginResult.Status.OK)); this.callbackContext.success();
} }
else if (action.equals("injectScriptCode")) { else if (action.equals("injectScriptCode")) {
String jsWrapper = null; String jsWrapper = null;
@ -311,11 +311,15 @@ public class InAppBrowser extends CordovaPlugin {
/** /**
* Closes the dialog * Closes the dialog
*/ */
private void closeDialog() { 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;
}
try { try {
final WebView childView = this.inAppWebView;
Runnable runnable = new Runnable() { Runnable runnable = new Runnable() {
@Override @Override
public void run() { public void run() {
childView.loadUrl("about:blank"); childView.loadUrl("about:blank");