neil@parkwaycc.co.uk Reporter
	Description • 19 years ago

I queried Bugzilla for instanceof ChromeWindow and found no hits, which
surprised me, as Mozilla 1.6 has this bug. Steps to reproduce problem:
1. From the browser, open the JS console
2. Evaluate top.opener.content instanceof ChromeWindow

Expected result: false

Actual result: true

Workaround: use Components.interfaces.nsIDOMChromeWindow

	Martijn Wargers (dead)
	Comment 1 • 19 years ago

It returns true for me now, with the 2005-08-27 trunk build. -> WFM?

	neil@parkwaycc.co.uk Reporter
	Comment 2 • 19 years ago

(In reply to comment #1)
>It returns true for me now, with the 2005-08-27 trunk build. -> WFM?
Comment #0 says Expected result: false

	Boris Zbarsky [:bzbarsky]
	Comment 3 • 19 years ago

This happens because sClassInfoData[class_name_struct->mDOMClassInfoID].mProtoChainInterface is nsIDOMWindow for chromewindow.

And that happens because the classinfo for ChromeWindow is:

  DOM_CLASSINFO_MAP_BEGIN_NO_CLASS_IF(ChromeWindow, nsIDOMWindow)

which means to use nsIDOMWindow as the proto chain interface.  Should we be using nsIDOMChromeWindow instead?  I'm not sure what else that proto chain interface affects...

	Peter Van der Beken [:peterv] Assignee
	Comment 4 • 15 years ago

Looks like this is fixed now. |alert(top.opener.content instanceof ChromeWindow)| alerts true for me. No idea what whould have fixed it.

	Dão Gottwald [:dao]
	Comment 5 • 15 years ago

(In reply to comment #4)
> Looks like this is fixed now. |alert(top.opener.content instanceof
> ChromeWindow)| alerts true for me.

It's expected to be false.

	Peter Van der Beken [:peterv] Assignee
	Comment 6 • 15 years ago

Sorry, got confused.

This works I think, need to think a bit more about it. Also need to add an automated testcase.

	Dão Gottwald [:dao]
	Comment 7 • 14 years ago

(In reply to comment #6)
> Created attachment 418021 [details] [diff] [review]
> v1
> 
> Sorry, got confused.
> 
> This works I think, need to think a bit more about it.

Have you thought about it? Any update?