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

Unified Diff: third_party/WebKit/Source/devtools/front_end/test_runner/TestRunner.js

Issue 2563553002: DevTools: Disallow console.log statements with eslint (Closed)
Patch Set: merge 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
Index: third_party/WebKit/Source/devtools/front_end/test_runner/TestRunner.js
diff --git a/third_party/WebKit/Source/devtools/front_end/test_runner/TestRunner.js b/third_party/WebKit/Source/devtools/front_end/test_runner/TestRunner.js
index e45f29df5ae2af9354d04dfd2caa86fcfcb5703c..c3ca762cc604b84d0a80ed806462d24e6e4af658 100644
--- a/third_party/WebKit/Source/devtools/front_end/test_runner/TestRunner.js
+++ b/third_party/WebKit/Source/devtools/front_end/test_runner/TestRunner.js
@@ -23,7 +23,7 @@ TestRunner._results = [];
*/
TestRunner.completeTest = function() {
if (!self.testRunner) {
- console.log('Test Done');
+ console.log('Test Done'); // eslint-disable-line no-console
chenwilliam 2016/12/08 20:00:23 Since it's a test harness, I think it's OK if we d
einbinder 2016/12/09 01:16:05 Done.
return;
}
@@ -51,7 +51,7 @@ TestRunner.addResult = function(text) {
if (self.testRunner)
TestRunner._results.push(String(text));
else
- console.log(text);
+ console.log(text); // eslint-disable-line no-console
};
/**

Powered by Google App Engine
This is Rietveld 408576698