The Cancel Debounce test case doesn't need to use callCount or startTime.
<rdar://problem/27479713>
<rdar://problem/27479718>
Created attachment 284261 [details] [Patch] Proposed Fix
Comment on attachment 284261 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=284261&action=review r=me! I could swear I had these review comments on the original! > LayoutTests/inspector/unit-tests/debounce.html:132 > - object.debounce(100).test(1, 'xyz'); > + object.debounce(100).test(); > object.test.cancelDebounce(); > > setTimeout(() => { InspectorTest.pass("Debounced function canceled."); resolve(); }, 200); It would be great to reduce these timeouts. 100 and 200 are very high for tests which typically run in 1-2 ms outside of LayoutTests/inspector. Want to change them to like 10 and 50?
Created attachment 284365 [details] [Patch] For Landing
Comment on attachment 284365 [details] [Patch] For Landing Clearing flags on attachment: 284365 Committed r203614: <http://trac.webkit.org/changeset/203614>
All reviewed patches have been landed. Closing bug.