tests/_base/window hasIE failure (IE9)

[bill]

    _AssertFailure: assertEqual() failed: 	expected
		7
	but got
		9

 with hint: 
		dojo.isIE should be 7 in withDoc w/ standards document w/ EmulateIE7
     ERROR IN: 		 function hasIE(t){
			var origIE = has("ie");
			win.withGlobal(fStandards.contentWindow, function(){
				t.is(7, dojo.isIE,"dojo.isIE should be 7 in withDoc w/ standards document w/ EmulateIE7");	// remove in 2.0
				t.is(7, has("ie"), "has('ie') should be 7 in withDoc w/ standards document w/ EmulateIE7");
			});
			t.is(origIE, dojo.isIE, "dojo.isIE should be reset to initial value");	// remove in 2.0
			t.is(origIE, has("ie"), "has('ie') should be reset to initial value");
		} FAILED test: ../../dojo/tests/_base/window.html::_base/window::hasIE 0 ms

[bill]

Probably a quirksmode failure due to that IE9 problem where type of outer window leaks into iframe. Happens in 1.7 too.

[Kris Zyp]

I am not sure I understand why a value of 7 is expected here. It looks like in dojo/_base/window.js, on lines 109-112 that we are explicitly overriding the version of IE based on the documentMode (which is 9 for the test). Should we not be doing that? That code came from [26334]. Is that when the regression occurred? Should we update the test?

[bill]

Well the test is loading window_iframe_standards.html, which has:

<meta http-equiv="X-UA-Compatible" content="IE=EmulateIE7">

Surely that means that for the iframe containing window_iframe_standards.html, has("ie") should return 7, document.documentMode should equal 7, and that the iframe should behave just like IE7.

The problem though (as I mentioned above) is that none of those things are happening, due to an IE9 bug where the settings from the outer frame leak into the iframe. I don't know any workaround to that bug.

Bottom line, probably that test should be disabled for IE9+.

PS: didn't mean to mark this as component=Query, I just got confused with #15799.

[Kris Zyp]

In [29441]:

Remove IE7 emulation tests that aren't really possible in IE9, fixes #15800 !strict

[bill]

In [30345]:

Disable IE7 emulation tests on IE9 since they aren't really possible in IE9, fixes #15800 on 1.7

[bill]

Note that test doesn't fail in 1.6 since the tests there are very different (ie there's no window.html, just window.js).