Bug 201083

Summary: Regression(r248533) Assertion hit in isMainThread() for some clients using WTF because the main thread is not initialized
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: Web Template FrameworkAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, beidson, benjamin, cmarcelo, commit-queue, dbates, ews-watchlist, fujii.hironori, ggaren, jeffm, rniwa, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=201947
Bug Depends on:    
Bug Blocks: 200507    
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2019-08-23 12:09:14 PDT
An assertion is hit in isMainThread() for some clients using WTF because the main thread is not initialized, since r248533.
Attachments
Patch (4.29 KB, patch)
2019-08-23 12:11 PDT, Chris Dumez
no flags
Chris Dumez
Comment 1 2019-08-23 12:10:30 PDT
Clients can work around this by calling WTF::initializeMainThread() before using WTF but it seems unfortunate to force them to do so. I propose we disable the assertion until the main thread is initialized.
Chris Dumez
Comment 2 2019-08-23 12:11:53 PDT
Chris Dumez
Comment 3 2019-08-23 12:14:40 PDT
WebKit Commit Bot
Comment 4 2019-08-23 13:31:15 PDT
Comment on attachment 377148 [details] Patch Clearing flags on attachment: 377148 Committed r249064: <https://trac.webkit.org/changeset/249064>
WebKit Commit Bot
Comment 5 2019-08-23 13:31:16 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2019-08-23 13:32:17 PDT
Fujii Hironori
Comment 7 2019-08-25 18:52:31 PDT
Chris Dumez
Comment 8 2019-08-26 09:16:44 PDT
(In reply to Fujii Hironori from comment #7) > Committed r249092: <https://trac.webkit.org/changeset/249092> Thanks for the build fox :D
Note You need to log in before you can comment on or make changes to this bug.