[Gnash-commit] gnash ChangeLog testsuite/actionscript.all/deja...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/02/09 14:47:15

Modified files:
        .              : ChangeLog 
        testsuite/actionscript.all: dejagnu_so_init.as 

Log message:
                * testsuite/actionscript.all/dejagnu_so_init.as: made use
                  of frame loop a compile-time define for having another way
                  to test setInterval.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.2290&r2=1.2291
http://cvs.savannah.gnu.org/viewcvs/gnash/testsuite/actionscript.all/dejagnu_so_init.as?cvsroot=gnash&r1=1.11&r2=1.12

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.2290
retrieving revision 1.2291
diff -u -b -r1.2290 -r1.2291
--- ChangeLog   9 Feb 2007 14:42:00 -0000       1.2290
+++ ChangeLog   9 Feb 2007 14:47:15 -0000       1.2291
@@ -1,5 +1,8 @@
 2007-02-09 Sandro Santilli <address@hidden>
 
+       * testsuite/actionscript.all/dejagnu_so_init.as: made use
+         of frame loop a compile-time define for having another way
+         to test setInterval.
        * testsuite/misc-ming.all/: Makefile.am, intervalTest.as,
          intervalTestRunner.cpp: add test *runner* for intervalTest.swf
 

Index: testsuite/actionscript.all/dejagnu_so_init.as
===================================================================
RCS file: /sources/gnash/gnash/testsuite/actionscript.all/dejagnu_so_init.as,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -b -r1.11 -r1.12
--- testsuite/actionscript.all/dejagnu_so_init.as       18 Dec 2006 12:57:54 
-0000      1.11
+++ testsuite/actionscript.all/dejagnu_so_init.as       9 Feb 2007 14:47:15 
-0000       1.12
@@ -1,4 +1,15 @@
 
+// frame loop seems not-working with SWF target 5, while
+// setInterval is not working with Gnash, we use both
+// to make sure it works :)
+//
+// Update: setInterval/clearInterval works now, so we
+//         don't need the frameloop anymore. We'll keep
+//        the define here just in case we want to make
+//        life easier for other free software players.
+//
+#define USE_FRAMELOOP
+
 // NOTE: when using ming-0.4.0-beta, a bug in 'makeswf' will
 //       prevent __shared_assets clip to work (the movieclip
 //       will be published with a frame-count of 0, thus
@@ -26,9 +37,11 @@
 checkIt = function() {
        if ( _root.dejagnu_module_initialized )
        {
-               // disable frameloop
-               onEnterFrame = undefined;
+               // disable loop
                clearInterval(_dejagnu_checker_interval);
+#ifdef USE_FRAMELOOP
+               onEnterFrame = undefined;
+#endif
 
                // setup some dejagnu wrappers
                info = function(msg) {
@@ -60,9 +73,11 @@
        }
        else if ( ++_dejagnu_checker_iterations > _dejagnu_checker_timeout )
        {
-               // disable frameloop
+               // disable loop
                clearInterval(_dejagnu_checker_interval);
+#ifdef USE_FRAMELOOP
                this.onEnterFrame = undefined;
+#endif
 
                // complain
                trace("No properly initialized dejagnu module found after "
@@ -145,6 +160,8 @@
 // to make sure it works :)
 
 _dejagnu_checker_interval = setInterval(checkIt, _dejagnu_checker_interval); 
+#ifdef USE_FRAMELOOP
 this.onEnterFrame = checkIt;
+#endif
 
 stop();