Summary: | Make SecurityOrigin safe to create and use from any thread | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | WebCore Misc. | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, darin, dbates, ews-watchlist, mkwst, webkit-bug-importer, youennf | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Chris Dumez
2018-03-31 19:09:53 PDT
Created attachment 336950 [details]
Patch
Created attachment 336951 [details]
Patch
Comment on attachment 336951 [details] Patch I wonder whether we should not be making more SchemeRegistry thread safe. All the methods only used by Document/DocumentLoader seem fine to me. I am less sure of the Database one but probably it is and will remain safe? Out of pure curiosity, it seems that we are really locking for write but maybe we could allow multiple read in parallel? Especially since we barely touch Schemes. View in context: https://bugs.webkit.org/attachment.cgi?id=336951&action=review > Source/WebCore/platform/SchemeRegistry.cpp:497 > { Ideally these registerXX should take a String&& since most of the time, they are called from IPC that can provide String&& Comment on attachment 336951 [details] Patch Clearing flags on attachment: 336951 Committed r230205: <https://trac.webkit.org/changeset/230205> All reviewed patches have been landed. Closing bug. |