Bug 109409

Using smart pointers (WTF::OwnPtr in this case) for Ecore_Timer is a bad idea because the timer handle becomes invalid as soon as the timer's callback returns ECORE_CALLBACK_CANCEL. ecore_timer_del() should NOT be called on an invalid timer handle. This is used in RunLoopEfl.cpp and may lead to crashes such as: STDOUT: <empty> STDERR: 1 0x7f6e7bdf094f STDERR: 2 0x7f6e7eb6b4a0 STDERR: 3 0x7f6e77aa1f80 ecore_timer_del STDERR: 4 0x7f6e7be23a83 WTF::deleteOwnedPtr(_Ecore_Timer*) STDERR: 5 0x7f6e7bcd4612 WTF::OwnPtr<_Ecore_Timer>::clear() STDERR: 6 0x7f6e7bcd43a0 WTF::OwnPtr<_Ecore_Timer>::operator=(std::nullptr_t) STDERR: 7 0x7f6e7bcd40d8 WebCore::RunLoop::TimerBase::timerFired(void*) STDERR: 8 0x7f6e77aa23de _ecore_timer_expired_call STDERR: 9 0x7f6e77aa25ab _ecore_timer_expired_timers_call STDERR: 10 0x7f6e77a9f4b1 STDERR: 11 0x7f6e77a9fb47 ecore_main_loop_begin STDERR: 12 0x7f6e7bcd3ea7 WebCore::RunLoop::run() STDERR: 13 0x7f6e7fa7036d WebProcessMainEfl STDERR: 14 0x400804 main STDERR: 15 0x7f6e7eb5676d __libc_start_main STDERR: 16 0x400729 STDERR: LEAK: 1 WebPage or crash log for WebKitTestRunner (pid 7660): STDOUT: <empty> STDERR: 6 0x7fa347f8bf28 WTF::OwnPtr<_Ecore_Timer>::operator=(std::nullptr_t) STDERR: 7 0x7fa347f8bc60 WebCore::RunLoop::TimerBase::timerFired(void*) STDERR: 8 0x7fa343d543de _ecore_timer_expired_call STDERR: 9 0x7fa343d545ab _ecore_timer_expired_timers_call STDERR: 10 0x7fa343d514b1 STDERR: 11 0x7fa343d51b47 ecore_main_loop_begin STDERR: 12 0x7fa347f8ba2f WebCore::RunLoop::run() STDERR: 13 0x7fa34bd254f5 WebProcessMainEfl STDERR: 14 0x400804 main STDERR: 15 0x7fa34ae0b76d __libc_start_main STDERR: 16 0x400729