Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Unified Diff: third_party/WebKit/LayoutTests/fast/forms/associatedFormControls-leak-nodes.html

Issue 2569763006: Make an existing workaround a bit broader. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/forms/associatedFormControls-leak-nodes.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/associatedFormControls-leak-nodes.html b/third_party/WebKit/LayoutTests/fast/forms/associatedFormControls-leak-nodes.html
index f0743c3b91a9728b95449c09eca28e6ee4ae7ccf..d1eed4504ef6bd8aa7e1731d7d8dbb8edb458560 100644
--- a/third_party/WebKit/LayoutTests/fast/forms/associatedFormControls-leak-nodes.html
+++ b/third_party/WebKit/LayoutTests/fast/forms/associatedFormControls-leak-nodes.html
@@ -12,7 +12,7 @@ if (!window.internals) {
window.jsTestIsAsync = true;
var documentsBefore;
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() {
documentsBefore = window.internals.numberOfLiveDocuments();
@@ -22,12 +22,15 @@ if (!window.internals) {
document.body.removeChild(frame);
frame = null;
+ // FIXME(keishi): crbug.com/674194
asyncGC(function() {
- documentsAfter = window.internals.numberOfLiveDocuments();
+ asyncGC(function() {
+ documentsAfter = window.internals.numberOfLiveDocuments();
- // -1 is from removing frame itself.
- shouldBe('documentsBefore - 1', 'documentsAfter');
- finishJSTest();
+ // -1 is from removing frame itself.
+ shouldBe('documentsBefore - 1', 'documentsAfter');
+ finishJSTest();
+ });
});
});
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698