[Webkit-unassigned] [Bug 147107] Adding default implementation for JavaScript watchdog.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 22 10:53:12 PDT 2015


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

--- Comment #9 from Mark Lam <mark.lam at apple.com> ---
(In reply to comment #8)
> Comment on attachment 257262 [details]
> Patch
> 
> Is there any reason to have two implementations of this feature? Why not
> just one?

Darwin port uses GCD.  The default implementation being proposed creates a thread per WatchDog i.e. per VM.  I believe the GCD solution is more efficient for Darwin, but is unfortunately not always available for other ports.

The watchdog thread solution could be optimized to use only one thread, but that would effectively be like implementing a form of GCD.  If the other ports don't instantiate multiple VMs as a normal pattern of usage, then it may not be worth the effort to work on this optimization (which is why I didn't bring it up in my review).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150722/e05d52dc/attachment.html>


More information about the webkit-unassigned mailing list