Bug 141538 – WKActionSheetAssistant crashes with a Bad Access when the view it's initialized with is no longer in memory

• Component: WebKit2
• Status: RESOLVED CONFIGURATION CHANGED

[McLaren Stanley]

Created attachment 246475 [details]
This Sample App shows one way this bug manifests

Overview:
 WKActionSheetAssistent.mm holds an unsafe pointer to a UIView called "_view".
 However since WKActionSheet.mm is retained by the window's rootViewController and it's
 UIAlertActions retain the WKActionSheetAssistent.mm, the assistent can easily outlive
 the _view it's pointing to. As a result this example will crash with a BAD ACCESS. The most
 common stack trace that that exposes this issue for us is via the "willRotate" method on WKActionSheet.mm.

 I will attach a crash log with that particular trace along with the bug that I file.

 For the record, we don't dump the webview on a memory warning in our production app, but the views that
 are being deallococated in the real app are internal subviews of WKWebView that we can't access, (not necessarily
 memory pressure)

Reproduction Steps:
 1. Run the attached sample app
 2. Long-press the web link
 3. Leave ActionSheet open
 4. Send App Memory Warning
 5. Close ActionSheet

Actual Result:
 The App Crashes with Bad Access

Expected Result:
 The App shouldn't crash

Platform: 
 iOS 8.1.3

[McLaren Stanley]

Created attachment 246477 [details]
Production crash log.

[Radar WebKit Bug Importer]

<rdar://problem/19830665>