Bug 22328 – Worker object should implement WindowTimers interface as per HTML5 spec

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: WebCore JavaScript

[Dmitry Titov]

Workers should get setTimeout/clearTimeout/setInterval/clearInterval implemented on them. working on a patch.

[Dmitry Titov]

Created attachment 25304 [details]
Initial Refactoring of code

[Dmitry Titov]

Refactored timeout code (set/clearTimeout,set/clearInterval) in preparation for implementing them on Workers.
Most of implementation moved from JSDOMWindowBase to ScriptExecutionContext, including the storage for timers.
Timers are now instances of new class DOMTimer. 
ScriptExecutionContext has a virtual method 'executeTimoutCallback' which is called when a timer fires. Subclasses of SEC (Document and WorkerContext) implement this differently. For now, only Document has implementation - depending on whether a callback is a JS function or a string of JS code, it either delegates back to JSDOMGlobalObject that captured the function and arguments, or asks FrameLoader to execute the script.
ScheduledAction is now just a collection of data for timer callback.

[Alexey Proskuryakov]

Comment on attachment 25304 [details]
Initial Refactoring of code

Aside from code style issues, I have two comments. First, DOMTimer inherits from TimerBase, which is not thread safe, and doesn't work in non-main threads anyway (on Mac OS X, it uses CFRunLoopTimerCreate on main run loop). Second, clearAllTimeouts and related functions need to be modified to take advantage of DOMTimer being an ActiveDOMObject. Pausing and resuming timers will probably require support for pausing/resuming active DOM objects, which is needed anyway.

You may want to discuss this with someone who actually knows how timers used to work though!

[Dmitry Titov]

*** This bug has been marked as a duplicate of 22718 ***