[TIMOB-525] clearTimeout triggers the call instead of just clearing

Here is a reproducible http://gist.github.com/335391">test case.

Make it your app.js and run it.

How to reproduce:

1. Click "move"
   
2. move a row. This sets a timeout to fire in 20 seconds.
   
3. move another row within 20 seconds. The timeout will fire.

I realize this could have been a simpler test, but I knew this would reproduce the problem, since this is how I ran into it in the first place.

Here's a stupidly simple broken test case:

   var win = Titanium.UI.createWindow({ backgroundColor:'#fff' });
   win.open();
   var timer = setTimeout(function() { alert('You should never see me!'); }, 5000);
   clearTimeout(timer);
   

Here's a generalized workaround I came up with:

   function timeoutObject() { this.flag = true; }
   function mySetTimeout(callback, time) {
     var myCallback = callback;
     if (typeof callback == 'string') {

   

   
   
   
   

   myCallback = function() { eval(callback); };
   

   
   
   

     }
     var timerobj = new timeoutObject();
     setTimeout(function () { 
   
   

   if (timerobj.flag) { myCallback(); }
   

   
   
   

     }, time);
     return timerobj;
   }
   function myClearTimeout(timer) {
     timer.flag = false;
   }
   

(from [3f94bd90743f2c323216909fe50a016a16ea6dcf]) Closes #525: Timers now stop before invocation if cancelled. http://github.com/appcelerator/titanium_mobile/commit/3f94bd90743f2c323216909fe50a016a16ea6dcf"> http://github.com/appcelerator/titanium_mobile/commit/3f94bd90743f2...