Revert of Consider any testharness result with >= 1 PASS and no FAIL to be passing.

third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/testharness_results_unittest.py

Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/testharness_results_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/testharness_results_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/testharness_results_unittest.py
index 2c78e0cc047abbde89aa09cb264c51fb609f58d2..1afd2ad774a2547b5ea325fde6ce42d890cd1e4d 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/testharness_results_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/testharness_results_unittest.py
@@ -9,22 +9,15 @@ from webkitpy.layout_tests.models import testharness_results
 
 class TestHarnessResultCheckerTest(unittest.TestCase):
 
-    def test_is_all_pass_testharness_result_positive_cases(self):
-        self.assertTrue(testharness_results.is_all_pass_testharness_result(
-            'This is a testharness.js-based test.\n'
-            ' PASS: foo bar \n'
-            ' Harness: the test ran to completion.'))
-        self.assertTrue(testharness_results.is_all_pass_testharness_result(
-            'This is a testharness.js-based test.\n'
-            'PASS \'grid\' with: grid-template-areas: "a b"\n'
-            '"c d";\n'
-            'Harness: the test ran to completion.\n'))
-
-    def test_is_all_pass_testharness_result_negative_cases(self):
+    def test_is_all_pass_testharness_result(self):
         self.assertFalse(testharness_results.is_all_pass_testharness_result(
             'This is a testharness.js-based test.\n'
             'CONSOLE WARNING: This is a warning.\n'
             'Test ran to completion.'))
+        self.assertTrue(testharness_results.is_all_pass_testharness_result(
+            'This is a testharness.js-based test.\n'
+            ' PASS: foo bar \n'
+            ' Harness: the test ran to completion.'))
         self.assertFalse(testharness_results.is_all_pass_testharness_result(
             'This is a testharness.js-based test.\n'
             ' PASS: foo bar \n'
@@ -64,12 +57,11 @@ class TestHarnessResultCheckerTest(unittest.TestCase):
             'Harness: the test ran to completion.'))
 
     def test_is_testharness_output_passing_empty_content(self):
-        self.assertFalse(testharness_results.is_testharness_output_passing(
+        self.assertTrue(testharness_results.is_testharness_output_passing(
             'This is a testharness.js-based test.\n'
             '   Harness: the test ran to completion.'))
 
-    def test_is_testharness_output_passing_no_pass(self):
-        # If there are no PASS lines, then the test is not considered to pass.
+    def test_is_testharness_output_passing_unexpected_content(self):
         self.assertFalse(testharness_results.is_testharness_output_passing(
             'This is a testharness.js-based test.\n'
             '  \n'
@@ -78,13 +70,11 @@ class TestHarnessResultCheckerTest(unittest.TestCase):
             'This is a testharness.js-based test.\n'
             ' Foo bar \n'
             ' Harness: the test ran to completion.'))
-
-    def test_is_testharness_output_passing_with_pass_and_random_text(self):
-        self.assertTrue(testharness_results.is_testharness_output_passing(
+        self.assertFalse(testharness_results.is_testharness_output_passing(
             'RANDOM TEXT.\n'
             'This is a testharness.js-based test.\n'
             'PASS: things are fine.\n'
-            ' Harness: the test ran to completion.\n'
+            '.Harness: the test ran to completion.\n'
             '\n'))
 
     def test_is_testharness_output_passing_basic_examples(self):
@@ -107,14 +97,13 @@ class TestHarnessResultCheckerTest(unittest.TestCase):
             ' Harness: the test ran to completion.'))
 
     def test_is_testharness_output_passing_with_console_messages(self):
-        self.assertFalse(testharness_results.is_testharness_output_passing(
+        self.assertTrue(testharness_results.is_testharness_output_passing(
             'This is a testharness.js-based test.\n'
             ' CONSOLE ERROR: BLAH  \n'
             ' Harness: the test ran to completion.'))
         self.assertTrue(testharness_results.is_testharness_output_passing(
             'This is a testharness.js-based test.\n'
             ' CONSOLE WARNING: BLAH  \n'
-            'PASS: some passing method\n'
             ' Harness: the test ran to completion.'))
         self.assertTrue(testharness_results.is_testharness_output_passing(
             'CONSOLE LOG: error.\n'