Don't count SLOW tests towards failure.

media/tools/layout_tests/layouttest_analyzer_helpers.py

 # Copyright (c) 2012 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.
 
 """Helper functions for the layout test analyzer."""
 
 from datetime import datetime
 from email.mime.multipart import MIMEMultipart
 from email.mime.text import MIMEText
 import fileinput
@@ skipping 45 matching lines @@
           refer to |test_expectaions.ALL_TE_KEYWORDS|).
     """
     self.result_map = {}
     self.result_map['whole'] = {}
     self.result_map['skip'] = {}
     self.result_map['nonskip'] = {}
     if test_info_map:
       for (k, value) in test_info_map.iteritems():
         self.result_map['whole'][k] = value
         if 'te_info' in value:
-          if any([True for x in value['te_info'] if 'SKIP' in x]):
+          if any([True for x in value['te_info']
+                  if 'SKIP' in x or 'SLOW' in x]):

[Ami GONE FROM CHROMIUM, 2012/08/08 20:32:27]

A SLOW line that has anything to the right of the = other than PASS should still
ding us, though.

[DaleCurtis, 2012/08/08 20:53:54]

Just PASS? Or PASS + modifiers? There are several lines that have "SLOW PASS
TIMEOUT|TEXT|etc" Although it looks like only video-stall-play is SLOW+PASS at
the moment.

[Ami GONE FROM CHROMIUM, 2012/08/08 20:57:09]

If there's anything other than PASS to the right of the '=' then that indicates
we still have a problem that needs fixing.  So it should count against our
failure %age.

[DaleCurtis, 2012/08/08 23:47:25]

Done. Although no tests now qualify for SLOW currently.

             self.result_map['skip'][k] = value
           else:
             self.result_map['nonskip'][k] = value
 
   @staticmethod
   def GetDiffString(diff_map_element, type_str):
     """Get difference string out of diff map element.
 
     The difference string shows difference between two analyzer results
     (for example, a result for now and a result for sometime in the past)
@@ skipping 513 matching lines @@
           list2 = map2[name]['te_info']
           te_diff = [item for item in list1 if not item in list2]
           if te_diff:
             name_list.append((name, te_diff))
       else:
         name_list.append((name, value1))
     return name_list
 
   return (GetDiffBetweenMapsHelper(map1, map2, lookIntoTestExpectationInfo),
           GetDiffBetweenMapsHelper(map2, map1, lookIntoTestExpectationInfo))