Change UIWebView to WKWebView

[rafaelpereznajera]

Better performance, better memory management ...

[mikerz]

I'm not sure this is a good idea; WKWebview is iOS 8.0+ and has its own bugs to deal with.

I'd welcome a separate branch using WKWebview though.

[ronakvora]

@mikerz; while I agree that WKWebView still hasn't sorted itself out completely, it's required on iPad for PiP :.

[ronakvora]

You can also use UIWebView as a backup for < iOS 8. We do this often in our application.

[the-civilian]

+1

[ashoemaker]

👍

[cdrx]

The main problem with WKWebView (that makes this non-trivial to implement) is that stringByEvaluatingJavascriptFromString is now async, meaning that the external API for this module would need to change

[cm-hugog]

+1

[hmhv]

I tried to use WKWebView. and i found WKWebView version use only about 20% of memory to play a example video.

https://github.com/hmhv/YoutubePlayer-in-WKWebView

[zsszatmari]

How did you measure it? In case of WKWebView the actual heavy lifting is done in a separate process, not your app's process, so if you are looking at xcode's app metrics you are mistaken.

[hmhv]

@zsszatmari Thank you your comment. you are right. I used xcode's app metrics. but I think It is still good for app's memory address space.

[cdrx]

One thing to note with WKWebView: when your app is backgrounded, iOS can shutdown the WKWebView process to reclaim some memory but leave your app running.

There isn't any event to detect this happening so to be safe you should reset your web view whenever your app resumes.

[mhuusko5]

If using WKWebView, could get Mac support as well..

[light-bo]

+1