[Gnash-commit] gnash ChangeLog testsuite/misc-ming.all/Drawing...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/05/29 09:05:00

Modified files:
        .              : ChangeLog 
        testsuite/misc-ming.all: DrawingApiTestRunner.cpp 

Log message:
                * testsuite/misc-ming.all/DrawingApiTestRunner.cpp: reintruduce
                  the pixel-accurate test after Udo fix for #19828.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.3390&r2=1.3391
http://cvs.savannah.gnu.org/viewcvs/gnash/testsuite/misc-ming.all/DrawingApiTestRunner.cpp?cvsroot=gnash&r1=1.17&r2=1.18

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.3390
retrieving revision 1.3391
diff -u -b -r1.3390 -r1.3391
--- ChangeLog   29 May 2007 08:49:14 -0000      1.3390
+++ ChangeLog   29 May 2007 09:05:00 -0000      1.3391
@@ -1,3 +1,8 @@
+2007-05-29 Sandro Santilli <address@hidden>
+
+       * testsuite/misc-ming.all/DrawingApiTestRunner.cpp: reintruduce
+         the pixel-accurate test after Udo fix for #19828.
+
 2007-05-29 Udo Giacomozzi <address@hidden>
 
        * backend/render_handler_agg.cpp: Fix coordinate translation and  

Index: testsuite/misc-ming.all/DrawingApiTestRunner.cpp
===================================================================
RCS file: 
/sources/gnash/gnash/testsuite/misc-ming.all/DrawingApiTestRunner.cpp,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -b -r1.17 -r1.18
--- testsuite/misc-ming.all/DrawingApiTestRunner.cpp    28 May 2007 15:41:12 
-0000      1.17
+++ testsuite/misc-ming.all/DrawingApiTestRunner.cpp    29 May 2007 09:05:00 
-0000      1.18
@@ -118,15 +118,10 @@
        // Over the black "hairlined" line
        tester.movePointerTo(250, 180);
        check(tester.isMouseOverMouseEntity());
-       // The line used to be rendered at Y=180.
-       // After pixel hinting was touched (see bug #19775)
-       // it moved to Y=179. Since isMouseOverMouseEntity()
-       // would return false for 250,180 we're going to
-       // tolerate the drift of one pixel for the line.
-       // This will accept both 179 and 180 as valid positions
-       // for the line. In the future we might add test for 
-       // exact placement using a more focused testcase.
-       // See bug #19828 for more info about this change.
+       // pixel at 250,180 is black
+       check_pixel(250, 180, 1, black, 2);
+       // pixels above and below 180 is white, so 
+       // we get gray with a radius of 2
        check_pixel(250, 180, 2, gray, 2);
 
        // Over the transparent line (150,100)