[Webkit-unassigned] [Bug 159980] [iOS] Apps using WKWebView will crash if they set the scroll view's delegate and don't nil it out later

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=159980

--- Comment #2 from mitz at webkit.org <mitz at webkit.org> ---
Comment on attachment 284134
  --> https://bugs.webkit.org/attachment.cgi?id=284134
Patch for [iOS] Apps using WKWebView will crash if they set the scroll views

I believe that the fundamental problem is that WKScrollView is subclassing UIScrollView and violating its API contract that states that the delegate property is weak. Your patch prevents this from causing a crash in one specific scenario, but I don’t understand why we can’t just fix WKScrollView. It may be as simple as using a WeakObjCPtr for the external delegate (making sure to take a strong reference any time we use it in a method). Am I missing anything?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160720/5aaf394c/attachment.html>