[JSC] DeferredWorkTimer should clear pending task after running
Created attachment 418210 [details] Patch
Created attachment 418211 [details] Patch
Comment on attachment 418211 [details] Patch Looking.
Created attachment 418219 [details] Patch
Created attachment 418220 [details] Patch
Comment on attachment 418220 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=418220&action=review r=me > Source/JavaScriptCore/runtime/JSPromise.cpp:-186 > - vm.deferredWorkTimer->cancelPendingWork(this); Is there a reason to still keep DeferredWorkTimer::cancelPendingWork()? I see that you've removed all calls to it. If not, we should remove it.
Comment on attachment 418220 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=418220&action=review >> Source/JavaScriptCore/runtime/JSPromise.cpp:-186 >> - vm.deferredWorkTimer->cancelPendingWork(this); > > Is there a reason to still keep DeferredWorkTimer::cancelPendingWork()? I see that you've removed all calls to it. If not, we should remove it. There are no users, so we can remove it :).
Committed r271781: <https://trac.webkit.org/changeset/271781>
<rdar://problem/73539704>