Updated V8 versions of ScheduledAction and WorkerContextCustom to actually invoke timers from workers in Chromium.
Created attachment 30602 [details] Proposed patch
Comment on attachment 30602 [details] Proposed patch Fine to land after addressing these style nits. > + } else { > + scriptController->evaluate(m_code); > + } No braces here? > + v8::Handle<v8::Value> function = args[0]; > + int32_t timeout = argumentCount >= 2 ? args[1]->Int32Value() : 0; Weird spacing thing?
Fixed according to Dimitri's comments and landed: http://trac.webkit.org/changeset/44160
Chromium fix landed as http://trac.webkit.org/changeset/44202. I should've picked this up during review :-\.