cordova-plugin-device
cordova-plugin-device copied to clipboard
fix(android): access Context through webView.getContext() (CB-12623)
Android WebView can be created without an Activity. One use case is to create a WebView within a Service and attach/detach it to an Activity window on demand.
Cordova CI Build has completed successfully.
Commit - Link Dashboard - Link
66 tests run, 0 skipped, 0 failed.
The CordovaWebView object is not actually an Android WebView, so this shouldn't be done.
@infil00p should we close this?
@filmaj Not sure. I'm not sure what the reporter is trying to do here.
@dreifachstein, could you elaborate please?
Looks good to me, but you'll have to update engines in package.json and plugin.xml to require cordova-android >= 7.0.0 as this is not available in older versions https://github.com/apache/cordova-android/commit/17906735dfe23e74cbe75394a0de3bf448233f25
And we on next plugin update should do a major bump because of the engines update
Let there be tests.
I didn't see the engine requirement cordova-android >= 7.0.0 being added as requested.
Will we add it after this PR is merged?
yeah, we can add it in a separate PR