RESOLVED FIXED 136804
Should call [BKSProcessAssertion invalidate] 
https://bugs.webkit.org/show_bug.cgi?id=136804
Summary Should call [BKSProcessAssertion invalidate]
Gavin Barraclough
Reported 2014-09-13 09:55:34 PDT
<rdar://problem/17897780> We should invalidate the process assertion before releasing it. Fortunately this is really easy, since there is a simple 1:1 relationship with ProcessAssertion, with their lifetimes matching exactly.
Attachments
Fix (3.03 KB, patch)
2014-09-13 09:58 PDT, Gavin Barraclough 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2014-09-13 09:58:22 PDT
Created attachment 238087 [details] Fix
Sam Weinig
Comment 2 2014-09-13 11:49:53 PDT
Why do we need to do this? What does it fix?
Gavin Barraclough
Comment 3 2014-09-13 18:52:07 PDT
(In reply to comment #2) > Why do we need to do this? What does it fix? See <rdar://problem/17897780>.
David Kilzer (:ddkilzer)
Comment 4 2014-09-15 09:29:46 PDT
(In reply to comment #2) > Why do we need to do this? What does it fix? It fixes a race condition that can cause a crash when closing a tab in MobileSafari.
Gavin Barraclough
Comment 5 2014-09-15 10:10:37 PDT
Committed revision 173623.
mitz
Comment 6 2014-09-15 10:28:49 PDT
I wish this bug (and the ChangeLog entry) had a title that explained why we were doing this. Something like “Safari crashed when closing a tab in -[BKSProcessAssertionClient _handleDestroy:]”.
Note You need to log in before you can comment on or make changes to this bug.