Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix android.view.WindowManager$BadTokenException crashes #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zsoltboldizsar
Copy link

Connected issue #31

The fix consists of checking whether the host activity, where the dialog is about to be shown, is still running and only proceed if it is, hence avoiding the crash from happening.

@ccorcos
Copy link

ccorcos commented Jun 12, 2018

I will test this out in the next week or two.

@ccorcos
Copy link

ccorcos commented Aug 2, 2018

The default promptInterval was changed from 10 to 1 btw.

int promptInterval = webView.getPreferences().getInteger("CRIPageLoadPromptInterval", 1);

@zsoltboldizsar
Copy link
Author

zsoltboldizsar commented Aug 2, 2018

Changed it back to its original value. Thanks for your time.

@ccorcos
Copy link

ccorcos commented Aug 6, 2018

Its in production. No more crashes :)

cvanderschuere added a commit to indie-technologies/cordova-plugin-remote-injection that referenced this pull request Feb 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants