Author: [email protected]
Date: Wed Jun 24 07:41:33 2009
New Revision: 5624
Modified:
releases/1.6/user/test/com/google/gwt/dom/client/ElementTest.java
Log:
Merging trunk r5492 into releases/1.6
Modified: releases/1.6/user/test/com/google/gwt/dom/client/ElementTest.java
==============================================================================
--- releases/1.6/user/test/com/google/gwt/dom/client/ElementTest.java
(original)
+++ releases/1.6/user/test/com/google/gwt/dom/client/ElementTest.java Wed
Jun 24 07:41:33 2009
@@ -105,14 +105,23 @@
div.getStyle().setPropertyPx("left", 1000);
div.getStyle().setPropertyPx("top", 1000);
+ // Get the absolute position of the element when the body is
unscrolled.
int absLeft = div.getAbsoluteLeft();
int absTop = div.getAbsoluteTop();
+ // Scroll the body as far down and to the right as possible.
body.setScrollLeft(10000);
body.setScrollTop(10000);
- assertEquals(absLeft, div.getAbsoluteLeft());
- assertEquals(absTop, div.getAbsoluteTop());
+ // Make sure the absolute position hasn't changed (this has turned out
to
+ // be a common error in getAbsoluteLeft/Top() implementations).
+ //
+ // HACK: Firefox 2 has a bug that causes its getBoxObjectFor() to
become
+ // off-by-one at times when scrolling. It's not clear how to make this
go
+ // away, and doesn't seem to be worth the trouble to implement
+ // getAbsoluteLeft/Top() yet again for FF2.
+ assertTrue(Math.abs(absLeft - div.getAbsoluteLeft()) <= 1);
+ assertTrue(Math.abs(absTop - div.getAbsoluteTop()) <= 1);
}
/**
--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---
