Closed
Bug 912797
Opened 11 years ago
Closed 11 years ago
rename Status to WorkerStatus
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: froydnj, Unassigned)
Details
Attachments
(1 file)
|
13.39 KB,
patch
|
Details | Diff | Splinter Review |
Turns out X11 typedef's Status. Depending on where headers are included, this can break rather a lot of things.
|
Reporter | |
Comment 1•11 years ago
|
||
I know we have gnarly namespaces for workers, but that's not good enough in this case. We also have mozilla::dom::workers::Worker$(FOO) for various other values of $(FOO), so I don't feel too awful about the workers::Worker ugliness
Attachment #799882 -
Flags: review?(khuey)
|
|
Reporter | |
Comment 2•11 years ago
|
||
Don't know if this is still needed once bug 909642 gets fixed. Would be interested to know if it still fixes your issues, Kyle.
I didn't actually need this after bug 909642.
|
|
Reporter | |
Updated•11 years ago
|
Attachment #799882 -
Flags: review?(khuey)
|
|
Reporter | |
Comment 4•11 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #3) > I didn't actually need this after bug 909642. OK then!
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
|
Assignee | |
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•