-
-
Notifications
You must be signed in to change notification settings - Fork 339
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
Support for WKWebView #90
Comments
You'll need to wait for the Cordova iOS platform 4.0, which allows to choose the type of webview required. The plugin will need to be updated, but changes are minimal, I made them here: https://github.com/saghul/cordova-plugin-iosrtc/tree/ios-platform-4 |
You can also use this for the interim: https://github.com/Telerik-Verified-Plugins/WKWebView |
Would be something like this enough to achieve the goal of this issue? |
See my patch. That does it, but 4.0.0 is not out yet.
|
Nide, so let's wait for 4.0.0 before you send such a PR :) |
Looks like 4.0.x is out. https://github.com/apache/cordova-ios/releases/tag/4.0.1 |
@saghul Got time to send a PR for this now that 4.0 is out? 🍾 |
Oh, looks like it was released 8 december, indeed: https://cordova.apache.org/announcements/2015/12/08/cordova-ios-4.0.0.html I'll make a PR soon. |
I updated my branch (https://github.com/saghul/cordova-plugin-iosrtc/tree/ios-platform-4) so it compiles now, but my app is behaving weirdly, so I haven't been able to test this properly yet. |
This is completely bizarre, none of my plugins load and the debugger gets disconnected automagically after updating to iOS platform 4.0.1. Sigh. |
Ok, managed to launch the app with a WKWebView and the plugin, but now it crashes when trying to make a call, somewhere inside the iosrtc plugin. Fun times ahead, I guess. |
Does such a crash just happen with your "WKWebView" based fork? |
Yep. |
happy christmas anyway! |
Good / bad news: the crash also happens when NOT using a WKWebView, so it seems related to the iOS platform upgrade to 4.0.0. |
Are you 100% sure it is not a fucking "memory warning" upgraded to "iOS killed me due to memory usage"? I've seen TONS of them, and usually they are not related to the iosrtc plugin, but to the usage of some cool CSS transition effect or too much loaded images. |
Yeah, pretty sure. I get some EXC_BAD_ACCESS when doing a getUserMedia, on the first line in saveMediaStream. |
You are not calling that within a WebSocket event, right? |
No, though I do for incoming calls. I do have the hack applied though. It works right now, so I'm trying to figure out what the problem could be... |
Just FYI: this is also being re-worked and close to completion here. Both UIWebView and WKWebView seem to be working, including fixes for the HTML/transparency trick... |
@telemakhos thanks for that useful link. It seems that some work must be done in order to use Cordova iOS 4.0.0. @mark-veenstra I cannot spend time on this. @saghul already did some work. The remaining work seems doable by following same steps as those done here. Pull requests are welcome in the meanwhile. |
@mark-veenstra if you look at the PR I sent, the changes required to make it build are very small. Unfortunately I have no clue why it randomly crashed. But it looked similar to #144. Feel free to apply saghul@90a7c35 on top of master and try to debug it... |
I have applied the patch from @saghul to the current master (which includes the fix). But I still encounter a I have read the steps from here, but I don't see where it is used in iosrtc. Any ideas? EDIT: |
The plugin has been updated to Cordova iOS 4, so let's this game continue. |
@ibc I believe you can close this one. |
OK, so this will be available in 3.0.0. |
The next generation of IOS browser is out:
https://developer.apple.com/library/ios/documentation/WebKit/Reference/WKWebView_Ref/
Does this plugin support this type of webview (instead of UIView)? Or can we at least coexist (i.e. if we run cordova within wkwebview instead of UIView?)
Thanks
The text was updated successfully, but these errors were encountered: