[Webkit-unassigned] [Bug 151391] [GLIB] Implement garbage collector timers

[bugzilla-daemon at webkit.org]

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

--- Comment #12 from Zan Dobersek <zan at falconsigh.net> ---
Comment on attachment 265749
  --> https://bugs.webkit.org/attachment.cgi?id=265749
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265749&action=review

As the next step, WebCore::GCController should be updated so that PLATFORM(GTK) matches USE(CF), primarily to move the GCController's timer away from MainThreadSharedTimer.

> Source/JavaScriptCore/heap/HeapTimer.cpp:171
> +        static_cast<HeapTimer*>(userData)->timerDidFire();

What do you think of just writing out ::timerDidFire() in this lambda?

> Source/JavaScriptCore/heap/IncrementalSweeper.cpp:75
> +    g_source_set_ready_time(m_timer.get(), g_get_monotonic_time() + (sweepTimeSlice * sweepTimeMultiplier));

You're adding 0.1 seconds to g_get_monotonic_time(), which returns microseconds.

-- 
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/20151123/5c2223b3/attachment.html>