Summary: | ErrorEvent / ProgressEvent should be exposed to workers | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | beidson, cdumez, commit-queue, darin, esprehn+autocc, kangil.han, kondapallykalyan, sam | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Chris Dumez
2016-06-09 20:55:40 PDT
Created attachment 280992 [details]
Patch
Comment on attachment 280992 [details]
Patch
Actually, I'm not sure. ErrorEvent::createForBindings uses an AtomicString, and can't that only be used from the main thread?
(In reply to comment #2) > Comment on attachment 280992 [details] > Patch > > Actually, I'm not sure. ErrorEvent::createForBindings uses an AtomicString, > and can't that only be used from the main thread? Using AtomicString in other threads is fine. What you cannot do is pass them to other threads. AtomicStrings are per thread AFAIK. (In reply to comment #3) > (In reply to comment #2) > > Comment on attachment 280992 [details] > > Patch > > > > Actually, I'm not sure. ErrorEvent::createForBindings uses an AtomicString, > > and can't that only be used from the main thread? > > Using AtomicString in other threads is fine. What you cannot do is pass them > to other threads. AtomicStrings are per thread AFAIK. This is correct - Each thread has its own AtomicString table, and its safe to use them within a given thread. Comment on attachment 280992 [details] Patch Clearing flags on attachment: 280992 Committed r201926: <http://trac.webkit.org/changeset/201926> All reviewed patches have been landed. Closing bug. |