as we have the HeapTimer implemented
Created attachment 161496 [details] the patch It shares most code with USE(CF). But I have to change CFTimeInterval to double. Geoffrey, is that OK?
Created attachment 161500 [details] the patch
Comment on attachment 161500 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=161500&action=review > Source/JavaScriptCore/heap/IncrementalSweeper.cpp:106 > + double elapsedTime = WTF::monotonicallyIncreasingTime() - sweepBeginTime; This patch looks good to me, Yong said he took double over CFTimeInterval here since all methods used return double's. Maybe some Apple folks can comment whether CFTimeInterval usage is absolutely needed here?
Comment on attachment 161500 [details] the patch LGTM.
Comment on attachment 161500 [details] the patch Clearing flags on attachment: 161500 Committed r127185: <http://trac.webkit.org/changeset/127185>
All reviewed patches have been landed. Closing bug.