Don't setTimeout() every time the stack is cleared via MessageChannel()

This commit is contained in:
Steve Gravrock
2017-06-16 09:58:41 -07:00
parent 2e737f50ca
commit 9ee7b4ee0b
3 changed files with 12 additions and 2 deletions

View File

@@ -1645,6 +1645,7 @@ getJasmineRequireObj().clearStack = function(j$) {
tail = tail.next = { task: fn };
channel.port2.postMessage(0);
} else {
currentCallCount = 0;
setTimeout(fn);
}
};

View File

@@ -42,7 +42,11 @@ describe("ClearStack", function() {
clearStack(function() { });
expect(setImmediate.calls.count()).toEqual(9);
expect(setTimeout).toHaveBeenCalled();
expect(setTimeout.calls.count()).toEqual(1);
clearStack(function() { });
expect(setImmediate.calls.count()).toEqual(10);
expect(setTimeout.calls.count()).toEqual(1);
});
it("uses MessageChannels when available", function() {
@@ -89,7 +93,11 @@ describe("ClearStack", function() {
clearStack(function() { });
expect(fakeChannel.port2.postMessage.calls.count()).toEqual(9);
expect(setTimeout).toHaveBeenCalled();
expect(setTimeout.calls.count()).toEqual(1);
clearStack(function() { });
expect(fakeChannel.port2.postMessage.calls.count()).toEqual(10);
expect(setTimeout.calls.count()).toEqual(1);
});
it("calls setTimeout when onmessage is called recursively", function() {

View File

@@ -32,6 +32,7 @@ getJasmineRequireObj().clearStack = function(j$) {
tail = tail.next = { task: fn };
channel.port2.postMessage(0);
} else {
currentCallCount = 0;
setTimeout(fn);
}
};