Commit 299b2474 authored by thakis's avatar thakis Committed by Commit bot

Make an existing workaround a bit broader.

Without this, the test seems to fail after clang roll
https://codereview.chromium.org/2577523002
Since there's already a workaround, this is likely triggering
the same existing bug that already has a workaround, and not
due to a new compiler bug.

BUG=673463,674194
TBR=tkent

Review-Url: https://codereview.chromium.org/2569763006
Cr-Commit-Position: refs/heads/master@{#438595}
parent 1f931462
...@@ -12,7 +12,7 @@ if (!window.internals) { ...@@ -12,7 +12,7 @@ if (!window.internals) {
window.jsTestIsAsync = true; window.jsTestIsAsync = true;
var documentsBefore; var documentsBefore;
var documentsAfter; var documentsAfter;
// FIXME(keishi): Calling asyncGC twice to fix flakiness. // FIXME(keishi): Calling asyncGC twice to fix flakiness, crbug.com/674194
asyncGC(function() { asyncGC(function() {
asyncGC(function() { asyncGC(function() {
documentsBefore = window.internals.numberOfLiveDocuments(); documentsBefore = window.internals.numberOfLiveDocuments();
...@@ -22,12 +22,15 @@ if (!window.internals) { ...@@ -22,12 +22,15 @@ if (!window.internals) {
document.body.removeChild(frame); document.body.removeChild(frame);
frame = null; frame = null;
// FIXME(keishi): crbug.com/674194
asyncGC(function() { asyncGC(function() {
documentsAfter = window.internals.numberOfLiveDocuments(); asyncGC(function() {
documentsAfter = window.internals.numberOfLiveDocuments();
// -1 is from removing frame itself. // -1 is from removing frame itself.
shouldBe('documentsBefore - 1', 'documentsAfter'); shouldBe('documentsBefore - 1', 'documentsAfter');
finishJSTest(); finishJSTest();
});
}); });
}); });
}); });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment