GTTF: Test launcher - correctly handle non-zero exit code with all tests passing

base/test/launcher/test_result.h

Index: base/test/launcher/test_result.h
diff --git a/base/test/launcher/test_result.h b/base/test/launcher/test_result.h
index cbae3d21ce249fbdc0337f435ac026ffae2ab404..b61cdd4b932ca42bfa35dbf539e21849eb54d118 100644
--- a/base/test/launcher/test_result.h
+++ b/base/test/launcher/test_result.h
@@ -14,12 +14,13 @@ namespace base {
 // Structure containing result of a single test.
 struct TestResult {
   enum Status {
-    TEST_UNKNOWN,  // Status not set.
-    TEST_SUCCESS,  // Test passed.
-    TEST_FAILURE,  // Assertion failure (think EXPECT_TRUE, not DCHECK).
-    TEST_TIMEOUT,  // Test timed out and was killed.
-    TEST_CRASH,    // Test crashed (includes CHECK/DCHECK failures).
-    TEST_SKIPPED,  // Test skipped (not run at all).
+    TEST_UNKNOWN,          // Status not set.
+    TEST_SUCCESS,          // Test passed.
+    TEST_FAILURE,          // Assertion failure (think EXPECT_TRUE, not DCHECK).
+    TEST_FAILURE_ON_EXIT,  // Test passed but executable exit code was non-zero.
+    TEST_TIMEOUT,          // Test timed out and was killed.
+    TEST_CRASH,            // Test crashed (includes CHECK/DCHECK failures).
+    TEST_SKIPPED,          // Test skipped (not run at all).
   };
 
   TestResult();
@@ -38,7 +39,9 @@ struct TestResult {
   // normally, possibly with an exit code indicating failure, but didn't crash
   // or time out in the middle of the test).
   bool completed() const {
-    return status == TEST_SUCCESS || status == TEST_FAILURE;
+    return status == TEST_SUCCESS ||
+        status == TEST_FAILURE ||
+        status == TEST_FAILURE_ON_EXIT;
   }
 
   // Full name of the test (e.g. "A.B").