RESOLVED INVALID 77003
Make publicURLManagerMap thread safe
https://bugs.webkit.org/show_bug.cgi?id=77003
Summary Make publicURLManagerMap thread safe
Kaustubh Atrawalkar
Reported 2012-01-25 03:41:28 PST
With reference to bug - https://bugs.webkit.org/show_bug.cgi?id=74386 there are few missing fixes needed to be done. They are as follows - 1. Replace back removed "NoStaticTables" from DOMURL.idl. needed to allow this to be used with workers. 2. The methods publicURLManagerMap and publicURLManager aren't threadsafe so they has potential race conditions since it can be called on Web Worker threads as well as the main thread. Make them thread-safe.
Attachments
Patch (2.28 KB, patch)
2012-01-25 03:45 PST, Kaustubh Atrawalkar
levin: review-
Kaustubh Atrawalkar
Comment 1 2012-01-25 03:45:10 PST
Created attachment 123915 [details] Patch Added AtomicallyInitializedStatic to make publicURLmanager thread safe. Replaced NoStaticTables.
David Levin
Comment 2 2012-01-25 10:07:54 PST
Kaustubh Atrawalkar
Comment 3 2012-02-07 00:55:57 PST
This issue is collected in https://bugs.webkit.org/show_bug.cgi?id=74386 by rolling out the patch and fixing the patch itself.
Note You need to log in before you can comment on or make changes to this bug.