incubator-callback-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Git at Apache <>
Subject incubator-cordova-android pull request: Added catch/throw around dialog dismiss
Date Tue, 21 Feb 2012 07:33:14 GMT
GitHub user obrand synchronized the pull request at

Added catch/throw around dialog dismiss

I have noticed that when going back the stack in the browser, often it causes a crash due
to the dialog being dismissed. This seems to be a well known issue in Android. For now I have
added a try/catch with no-op around the dismiss.

The typical exception is:

java.lang.IllegalArgumentException: View not attached to window manager
	at android.view.WindowManagerImpl.findViewLocked(
	at android.view.WindowManagerImpl.removeView(
	at android.view.WindowManagerImpl$CompatModeWrapper.removeView(
	at com.phonegap.DroidGap.spinnerStop(
	at com.phonegap.DroidGap$GapViewClient$1$
	at android.os.Handler.handleCallback(
	at android.os.Handler.dispatchMessage(
	at android.os.Looper.loop(
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(
	at dalvik.system.NativeStart.main(Native Method)

You can merge this pull request into a Git repository by running
$ git pull dialog

Alternatively you can review and apply these changes as the patch at

View raw message