[Webkit-unassigned] [Bug 25974] [V8] Enable timers in Chromium workers

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon May 25 10:30:48 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=25974


dglazkov at chromium.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30602|review?                     |review+
               Flag|                            |




------- Comment #2 from dglazkov at chromium.org  2009-05-25 10:30 PDT -------
(From update of attachment 30602)
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?


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list