Improve run-webkit-tests error reporting when ref tests have missing images.

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

Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
index 1ae771a928337e7f16c212686daf7e0dfcf6f5e1..0c23f06c700f4aa2944f7df8b885410a282f42c2 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
@@ -32,9 +32,14 @@ from webkitpy.layout_tests.models import test_expectations
 
 
 def is_reftest_failure(failure_list):
-    failure_types = [type(f) for f in failure_list]
-    return set((FailureReftestMismatch, FailureReftestMismatchDidNotOccur, FailureReftestNoImagesGenerated)).intersection(
-        failure_types)
+    input_failure_types = {type(f) for f in failure_list}
+    reftest_failure_types = {
+        FailureReftestMismatch,
+        FailureReftestMismatchDidNotOccur,
+        FailureReftestNoImageGenerated,
+        FailureReftestNoReferenceImageGenerated
+    }
+    return bool(input_failure_types & reftest_failure_types)
 
 # FIXME: This is backwards.  Each TestFailure subclass should know what
 # test_expectation type it corresponds too.  Then this method just
@@ -220,14 +225,24 @@ class FailureReftestMismatchDidNotOccur(TestFailure):
         return "reference mismatch didn't happen"
 
 
-class FailureReftestNoImagesGenerated(TestFailure):
+class FailureReftestNoImageGenerated(TestFailure):
+
+    def __init__(self, reference_filename=None):
+        super(FailureReftestNoImageGenerated, self).__init__()
+        self.reference_filename = reference_filename
+
+    def message(self):
+        return "reference test didn't generate pixel results"
+
+
+class FailureReftestNoReferenceImageGenerated(TestFailure):
 
     def __init__(self, reference_filename=None):
-        super(FailureReftestNoImagesGenerated, self).__init__()
+        super(FailureReftestNoReferenceImageGenerated, self).__init__()
         self.reference_filename = reference_filename
 
     def message(self):
-        return "reference didn't generate pixel results."
+        return "-expected.html didn't generate pixel results"
 
 
 class FailureMissingAudio(TestFailure):
@@ -255,6 +270,8 @@ ALL_FAILURE_CLASSES = (FailureTimeout, FailureCrash, FailureMissingResult,
                        FailureTextMismatch, FailureMissingImageHash,
                        FailureMissingImage, FailureImageHashMismatch,
                        FailureImageHashIncorrect, FailureReftestMismatch,
-                       FailureReftestMismatchDidNotOccur, FailureReftestNoImagesGenerated,
+                       FailureReftestMismatchDidNotOccur,
+                       FailureReftestNoImageGenerated,
+                       FailureReftestNoReferenceImageGenerated,
                        FailureMissingAudio, FailureAudioMismatch,
                        FailureEarlyExit)