Fix testharness result parsing, make "Harness error" count as failure.

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

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import unittest
 
 from webkitpy.layout_tests.models import testharness_results
 
 
 class TestHarnessResultCheckerTest(unittest.TestCase):
@@ skipping 12 matching lines @@
     def test_is_all_pass_testharness_result_negative_cases(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.assertFalse(testharness_results.is_all_pass_testharness_result(
             'This is a testharness.js-based test.\n'
             ' PASS: foo bar \n'
             'FAIL  \n'
             ' Harness: the test ran to completion.'))
+        self.assertFalse(testharness_results.is_all_pass_testharness_result(
+            'This is a testharness.js-based test.\n'
+            'Harness Error. harness_status.status = 1\n'
+            'PASS foo bar\n'
+            'Harness: the test ran to completion.'))
 
     def test_is_testharness_output_positive_cases(self):
         self.assertTrue(testharness_results.is_testharness_output(
             'This is a testharness.js-based test.\n'
             'Harness: the test ran to completion.'))
         self.assertTrue(testharness_results.is_testharness_output(
             '\n'
             ' \r This is a testharness.js-based test. \n'
             ' \r  \n'
             ' \rHarness: the test ran to completion.   \n'
@@ skipping 129 matching lines @@
         self.assertFalse(testharness_results.has_console_errors_or_warnings(
             'This is a testharness.js-based test.\n'
             'CONSOLE MESSAGE: This is not error.'))
         self.assertFalse(testharness_results.has_console_errors_or_warnings(
             'This is a testharness.js-based test.\n'
             'No errors here.'))
         self.assertFalse(testharness_results.has_console_errors_or_warnings(
             'This is not a CONSOLE ERROR, sorry.'))
         self.assertFalse(testharness_results.has_console_errors_or_warnings(
             'This is not a CONSOLE WARNING, sorry.'))