Bug 160140

Summary: [iOS] Make sure we call the ProcessAssertion invalidation handler on the main thread
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, barraclough, ddkilzer, sam, webkit-bug-importer
Priority: P1 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: iPhone / iPad   
OS: Unspecified   
Attachments:
Description Flags
Patch darin: review+

Description Chris Dumez 2016-07-23 20:13:47 PDT
Make sure we call the ProcessAssertion invalidation handler on the main thread. BKSProcessAssertion calls our invalidation handler on a background thread which leads to thread safety problems and crashes.
Comment 1 Chris Dumez 2016-07-23 20:14:04 PDT
<rdar://problem/27399998>
Comment 2 Chris Dumez 2016-07-23 20:32:52 PDT
Created attachment 284433 [details]
Patch
Comment 3 Darin Adler 2016-07-25 09:53:51 PDT
Comment on attachment 284433 [details]
Patch

Still need to update the iOS simulator half of ProcessAssertionIOS.mm to use WTF::Function instead of std::function.
Comment 4 Chris Dumez 2016-07-25 09:58:35 PDT
Committed r203687: <http://trac.webkit.org/changeset/203687>