Jesse Ruderman Reporter
	Description • 11 years ago

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

	Jesse Ruderman 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?

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 2 • 11 years ago

Worker implementation is very special, and we must convert it to use normal webidl/non-manual-JSAPI stuff.

	Boris Zbarsky [:bzbarsky]
	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?

	Jesse Ruderman Reporter
	Comment 4 • 11 years ago

Long-term, will converting Window to WebIDL (bug 789261) take care of this?

	Kyle Huey (Exited; not receiving bugmail, old account, do not use) 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.

	Jesse Ruderman Reporter
	Comment 6 • 11 years ago

Is there a bug for moving workers to WebIDL?  We can make this bug depend on it.

	Boris Zbarsky [:bzbarsky]
	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.

	Masatoshi Kimura [:emk]
	Comment 8 • 11 years ago

Workers have been converted to WebIDL.

	Kyle Huey (Exited; not receiving bugmail, old account, do not use) Assignee
	Comment 9 • 11 years ago

Want to check in the test?

	Jesse Ruderman 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.