search

readium / SDKLauncher-iOS

A small iOS application to serve as a launcher/testbed for the Readium SDK.
BSD 3-Clause "New" or "Revised" License
71 stars 47 forks source link

Remove WKWebView scriptMessageHandler at clean up time #83

Closed msintov closed 8 years ago

msintov commented 8 years ago

Fix potential EPubViewController leak by removing WKWebView's WKUserContentController script message handler in cleanUp(). The script message handler is set on WKUserContentController in EPubViewController::loadView.

danielweck commented 8 years ago

This looks like a no-brainer to me. I vote for merging.

rkwright commented 8 years ago

@danielweck , @msintov I am OK with merging.