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

Unified Diff: chrome/test/data/extensions/api_test/automation/tests/tabs/common.js

Issue 485843002: Add logspam to help diagnose flaky tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Log --> LOG Created 6 years, 4 months 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
Index: chrome/test/data/extensions/api_test/automation/tests/tabs/common.js
diff --git a/chrome/test/data/extensions/api_test/automation/tests/tabs/common.js b/chrome/test/data/extensions/api_test/automation/tests/tabs/common.js
index af464a2573a079b980d4cbcb2470a88c6610d6b7..5e41c98e8e3acd016f4bfb135e2c548b18ae31e6 100644
--- a/chrome/test/data/extensions/api_test/automation/tests/tabs/common.js
+++ b/chrome/test/data/extensions/api_test/automation/tests/tabs/common.js
@@ -24,10 +24,15 @@ function setUpAndRunTests(allTests) {
chrome.automation.getTree(function (returnedRootNode) {
rootNode = returnedRootNode;
if (rootNode.attributes.docLoaded) {
+ console.log('In getTree() callback: ' +
+ 'docLoaded attr already true; running tests');
chrome.test.runTests(allTests);
return;
}
+ console.log('In getTree() callback: ' +
+ 'docLoaded false; waiting for loadComplete');
rootNode.addEventListener('loadComplete', function() {
+ console.log('loadComplete received; running tests.');
chrome.test.runTests(allTests);
});
});

Powered by Google App Engine
This is Rietveld 408576698