Closed
Bug 899046
Opened 11 years ago
Closed 11 years ago
Object.getOwnPropertyNames(window) is missing Worker until it is used
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla28
People
(Reporter: jruderman, Assigned: khuey)
References
Details
(Keywords: testcase)
Attachments
(1 file)
|
(deleted),
text/html
|
Details |
Even with bug 807222 fixed, Object.getOwnPropertyNames(window) is still missing "Worker".
In fact, once I use Worker, several more properties appear:
WorkerEvent,WorkerMessageEvent,WorkerErrorEvent,WorkerProgressEvent
|
Reporter | |
Comment 1•11 years ago
|
||
This caused my fuzzer to miss bug 897678.
If these are the only missing properties, I can work around the problem by teaching my fuzzer about Worker. Are there likely to be others?
|
||
Comment 2•11 years ago
|
||
Worker implementation is very special, and we must convert it to use normal webidl/non-manual-JSAPI stuff.
|
||
Comment 3•11 years ago
|
||
Looks like workers are the main special-case here, yeah.
We could probably fix this by doing a ResolveWorkerClasses with the "Worker" id in nsWindowSH::Enumerate. Seems hacky, but would fix this bug... Worth doing?
Flags: needinfo?(khuey)
|
|
Reporter | |
Comment 4•11 years ago
|
||
Long-term, will converting Window to WebIDL (bug 789261) take care of this?
|
Assignee | |
Comment 5•11 years ago
|
||
Converting workers to webidl should be enough, I believe.
We can hack through it if needed but I'd rather just do that.
Flags: needinfo?(khuey)
|
|
Reporter | |
Comment 6•11 years ago
|
||
Is there a bug for moving workers to WebIDL? We can make this bug depend on it.
|
|
||
Comment 7•11 years ago
|
||
> Long-term, will converting Window to WebIDL (bug 789261) take care of this?
No, but converting Workers to WebIDL will like Kyle says.
|
||
Comment 8•11 years ago
|
||
Workers have been converted to WebIDL.
|
||
Updated•11 years ago
|
Assignee: nobody → khuey
Target Milestone: --- → mozilla28
|
|
Reporter | |
Comment 10•11 years ago
|
||
I don't think this needs its own test now that it just uses the same code as everything else.
Flags: needinfo?(jruderman)
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•