Bug 180449

Summary: Use WTF::RecursiveLockAdapter instead of using pthread_mutex_t with recursive lock option
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: New BugsAssignee: Yusuke Suzuki <ysuzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, darin, ews-watchlist, mark.lam, mmaxfield, sam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch mark.lam: review+

Description Yusuke Suzuki 2017-12-05 14:51:44 PST 
Use WTF::RecursiveLockAdapter instead of using pthread_mutex_t with recursive lock option
Comment 1 Yusuke Suzuki 2017-12-05 14:52:35 PST 
Created attachment 328505 [details]
Patch
Comment 2 Yusuke Suzuki 2017-12-07 04:49:29 PST 
Created attachment 328690 [details]
Patch
Comment 3 Mark Lam 2017-12-07 10:28:24 PST 
Comment on attachment 328690 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=328690&action=review

r=me

> Source/WebCore/ChangeLog:8
> +        Use WTF::RecursiveLockAdapter<StaticLock> instead. We can remove pthred_mutex_xxx,

typo: /pthred_mutex_xxx/pthread_mutex_xxx/
Comment 4 Yusuke Suzuki 2017-12-07 18:18:49 PST 
Comment on attachment 328690 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=328690&action=review

>> Source/WebCore/ChangeLog:8
>> +        Use WTF::RecursiveLockAdapter<StaticLock> instead. We can remove pthred_mutex_xxx,
> 
> typo: /pthred_mutex_xxx/pthread_mutex_xxx/

Thanks, fixed.
Comment 5 Yusuke Suzuki 2017-12-07 19:02:31 PST 
Committed r225667: <https://trac.webkit.org/changeset/225667>
Comment 6 Radar WebKit Bug Importer 2017-12-07 19:06:26 PST 
<rdar://problem/35926300>