Opened 8 years ago

Closed 8 years ago

#12446 closed defect (fixed)

tests.window.scroll failing on chrome 10

Reported by: bill Owned by: Douglas Hays
Priority: high Milestone: 1.6.1
Component: HTML Version: 1.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

Chrome 10 is getting three test failures for me in the tets.window.scroll test. (Presumably this worked with chrome 9 but I don't know how to get the old version of chrome to test.)

	_AssertFailure: doh._AssertFailure: assertEqual() failed:
	expected
		(-25,-25)
	but got
		(-0,-25)

 with hint: 
		absContent_loose_rtl max failed
../../../util/doh/_browserRunner.js:352	ERROR IN:
		 function test_absContent(){
					var minScrollX, minScrollY, maxScrollX, maxScrollY;
					if(dojo.isIE){
						maxScrollX = -10 - sb;
						maxScrollY = (dojo.isIE >= 8)? maxScrollX : (-38 - sb);
					}else{
						maxScrollX = maxScrollY = -10 - sb;
					}
					minScrollX = minScrollY = sb + 10;
					generateTestXY('absContent_strict', maxScrollX, maxScrollY, minScrollX, minScrollY);
					if(dojo.isIE < 7){
						maxScrollX -= (sb & 1); // odd sized sb adds 1 more
						minScrollX -= (sb & 1); // odd sized sb adds 1 more
					}
					if(dojo.isIE == 7){ minScrollX = -90; }
					if(dojo.isOpera){
						maxScrollX = 90;
					}
					generateTestXY('absContent_loose_rtl', maxScrollX, maxScrollY, minScrollX, minScrollY);
					if(dojo.isIE){
						maxScrollX = (dojo.isIE >= 7? -10 : -6) - sb;
						maxScrollY = (dojo.isIE >= 7? -38 : -34) - sb;
						minScrollY = minScrollX = -maxScrollX;
					}else{
						maxScrollX = maxScrollY = -(sb+10);
						minScrollX = minScrollY = sb+10;
					}
					generateTestXY('absContent_quirks', maxScrollX, maxScrollY, minScrollX, minScrollY);
					if(dojo.isIE < 8){
						maxScrollX = (dojo.isIE >= 7 ? -10 : -6) - sb - (sb & 1);
						minScrollX = "+" + ((dojo.isIE >= 7? 10 : 6) + sb - (sb & 1));
					}
					if(dojo.isOpera){
						maxScrollX = 90;
					}
					generateTestXY('absContent_quirks_rtl', maxScrollX, maxScrollY, minScrollX, minScrollY);
				}
../../../util/doh/_browserRunner.js:352	_AssertFailure: doh._AssertFailure: assertEqual() failed:
	expected
		(+0,-0)
	but got
		(-1,-0)

 with hint: 
		bottomFloated_loose_rtl max failed

../../../util/doh/_browserRunner.js:352	ERROR IN:
		 function test_bottomFloated(){
					generateTest('bottomFloated_strict', "(+0,-0)", "(+0,-0)");
					generateTest('bottomFloated_quirks', "(+0,-0)", "(+0,-0)");
					generateTest('bottomFloated_loose_rtl', "("+(dojo.isOpera? -1 : (((dojo.isIE >= 8 || dojo.isWebKit)? "+" : "-")+0))+",-0)", "("+(dojo.isOpera? (sb+20) : (((dojo.isIE < 8 || dojo.isMoz)?"-":"+")+0))+",-0)");
					generateTest('bottomFloated_quirks_rtl', "("+(dojo.isOpera? -1 : ((dojo.isWebKit? "+" : "-")+0))+",-0)", "("+(dojo.isOpera? (sb+20) : (((dojo.isIE || dojo.isMoz)?"-":"+")+0))+",-0)");
				}
../../../util/doh/_browserRunner.js:352	_AssertFailure: doh._AssertFailure: assertEqual() failed:
	expected
		(+25,+35)
	but got
		(-1,+35)

 with hint: 
		tooBig_loose_rtl max failed

../../../util/doh/_browserRunner.js:352	ERROR IN:
		 function test_tooBig(){
					var fudge = 0;
					if(dojo.isIE <= 6){
						fudge = dojo.byId('tooBig_quirks').contentWindow.document.body.clientHeight-100+sb; // needed for running inside DOH runner
					}
					var minScroll, maxScroll;
					generateTestXY('tooBig_strict', sb+10,sb+20, 1,1);
					generateTestXY('tooBig_quirks', sb+10-fudge,sb+20-fudge, 1,1);
					if(dojo.isIE <= 7 || dojo.isMoz){
						maxScroll = -1;
					}else if(dojo.isIE){
						maxScroll = +20;
					}else if(dojo.isOpera){
						maxScroll = -90;
					}else{
						maxScroll = sb+10;
					}
					if(dojo.isIE <= 7 || dojo.isMoz){
						minScroll = -20;
					}else if(dojo.isOpera){
						minScroll = 80 + sb;
					}else{
						minScroll = 1;
					}
					generateTestXY('tooBig_loose_rtl', maxScroll,sb+20, minScroll,1);
					if(dojo.isIE >= 8){
						maxScroll = -1;
						minScroll = -20;
					}
					generateTestXY('tooBig_quirks_rtl', maxScroll,sb+20-fudge, minScroll,1);
				}
../../../util/doh/_browserRunner.js:352FAILED test: test_tooBig 4 ms

Change History (5)

comment:1 Changed 8 years ago by bill

Summary: tesets.window.scroll failing on chrome 10tests.window.scroll failing on chrome 10

comment:2 Changed 8 years ago by Douglas Hays

Milestone: tbd1.6.1
Status: newassigned

comment:3 Changed 8 years ago by Douglas Hays

(In [24206]) Fixes #12477, #12446. Added support for FF4 having the scrollbar on the left in RTL mode. Added support for IE9 munging iframe behavior to match parent container DOCTYPE/compatMode. Tweaked test numbers for FF4/IE9/Chrome10. !strict

comment:4 Changed 8 years ago by Douglas Hays

(In [24207]) Refs #12477, #12446. Backport [24206] to 1.6.1. !strict

comment:5 Changed 8 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.