WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
216370
CSS parsing-related singletons should only be used on the main thread
https://bugs.webkit.org/show_bug.cgi?id=216370
Summary
CSS parsing-related singletons should only be used on the main thread
Chris Lord
Reported
2020-09-10 09:15:14 PDT
Currently, there are various NeverDestroyed singletons involved in CSS parsing that are not intended to ever be used off of the main thread. A sub-set of CSS parsing is required for full OffscreenCanvas support (colour and font strings), so it'd be good for these singletons to be correctly declared as MainThreadNeverDestroyed so that if they are accidentally ever used off of the main thread, we would start to get test failures in debug mode.
Attachments
Patch
(4.03 KB, patch)
2020-09-10 09:27 PDT
,
Chris Lord
no flags
Details
Formatted Diff
Diff
Patch
(3.37 KB, patch)
2020-09-11 01:54 PDT
,
Chris Lord
no flags
Details
Formatted Diff
Diff
Patch
(3.36 KB, patch)
2020-09-14 00:57 PDT
,
Chris Lord
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Chris Lord
Comment 1
2020-09-10 09:27:02 PDT
Created
attachment 408451
[details]
Patch
Chris Lord
Comment 2
2020-09-10 09:29:44 PDT
Note that I changed the RenderTheme singletons for Gtk/Adwaita to MainThread but I haven't touched other platforms as I don't know if they have the same restriction. I expect, perhaps bar iOS, they do, but I'd rather not make the change blind, so if this is accepted, we can open a separate bug/bugs to keep track of other platforms that this doesn't include. I do wonder if NeverDestroyed should default to asserting the creation thread and the exception should be allowing concurrent access, rather than this way round.
Chris Lord
Comment 3
2020-09-11 01:54:43 PDT
Created
attachment 408525
[details]
Patch
Chris Lord
Comment 4
2020-09-14 00:57:07 PDT
Created
attachment 408685
[details]
Patch
EWS
Comment 5
2020-09-14 01:38:58 PDT
Committed
r267009
: <
https://trac.webkit.org/changeset/267009
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 408685
[details]
.
Radar WebKit Bug Importer
Comment 6
2020-09-14 01:39:23 PDT
<
rdar://problem/68838178
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug