Closed
Bug 1084439
Opened 10 years ago
Closed 10 years ago
Replace ResolveWorkerClasses by appropriate [Exposed=System] annotations
Categories
(Core :: DOM: Workers, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
Details
Attachments
(1 file)
13.44 KB,
patch
|
bholley
:
review+
|
Details | Diff | Splinter Review |
Right now, the only caller is the backstagepass code, which is precisely what Exposed=System is for.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8507013 -
Flags: review?(bobbyholley)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Comment 2•10 years ago
|
||
Comment on attachment 8507013 [details] [diff] [review] Remove ResolveWorkerClasses in favor of [Exposed=System] annotations Review of attachment 8507013 [details] [diff] [review]: ----------------------------------------------------------------- \o/
Attachment #8507013 -
Flags: review?(bobbyholley) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/916278924d4c
Target Milestone: --- → mozilla36
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/916278924d4c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 5•10 years ago
|
||
Comment on attachment 8507013 [details] [diff] [review] Remove ResolveWorkerClasses in favor of [Exposed=System] annotations Review of attachment 8507013 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/workers/RuntimeService.cpp @@ +183,4 @@ > ID_ChromeWorker, > ID_Event, > ID_MessageEvent, > ID_ErrorEvent, Are these enum values still used?
Assignee | ||
Comment 6•10 years ago
|
||
No. They weren't used before either, really, except ID_Count. I'll remove them.
Assignee | ||
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/037d1a9bc26a to remove those.
You need to log in
before you can comment on or make changes to this bug.
Description
•