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

Unified Diff: tracing/tracing/base/headless_tests.html

Issue 2776653002: [ESLint] Fix violations when enabling curly rule in eslint. (Closed)
Patch Set: rebase Created 3 years, 9 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
« no previous file with comments | « tracing/tracing/base/guid.html ('k') | tracing/tracing/base/in_memory_trace_stream.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/base/headless_tests.html
diff --git a/tracing/tracing/base/headless_tests.html b/tracing/tracing/base/headless_tests.html
index f9b9975de26608b8cc50948e7f3eebd5de630156..7dbf551e15a886bf7c5347dc23b320f33c3282a9 100644
--- a/tracing/tracing/base/headless_tests.html
+++ b/tracing/tracing/base/headless_tests.html
@@ -17,10 +17,11 @@ tr.exportTo('tr.b.unittest', function() {
throw new Error('headless_tests.html only works in headless mode');
}
function quit(errCode) {
- if (tr.isVinn)
+ if (tr.isVinn) {
global.quit(errCode);
- else
+ } else {
process.exit(errCode);
+ }
}
function printSpacer() {
@@ -37,8 +38,9 @@ tr.exportTo('tr.b.unittest', function() {
p = p.then(
function didAllSuitesLoad() {
var tests = loader.getAllTests().filter(function(testCase) {
- if (testCase instanceof tr.b.unittest.PerfTestCase)
+ if (testCase instanceof tr.b.unittest.PerfTestCase) {
return false;
+ }
return true;
});
if (tests.length === 0) {
@@ -59,10 +61,11 @@ tr.exportTo('tr.b.unittest', function() {
p = p.then(
function didAllTestRun() {
- if (results.numTestsThatFailed > 0)
+ if (results.numTestsThatFailed > 0) {
quit(1);
- else
- quit(0);
+ } else {
+ quit(0);
+ }
},
function testHarnessError(e) {
console.log('FAILED: A test harness error has ocurred.');
« no previous file with comments | « tracing/tracing/base/guid.html ('k') | tracing/tracing/base/in_memory_trace_stream.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698