Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(224)

Side by Side Diff: Tools/Scripts/webkitpy/layout_tests/models/test_results.py

Issue 303223008: Overlay expected and actual repaint rects for LayoutTests. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add several small features Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright (C) 2010 Google Inc. All rights reserved. 1 # Copyright (C) 2010 Google Inc. All rights reserved.
2 # 2 #
3 # Redistribution and use in source and binary forms, with or without 3 # Redistribution and use in source and binary forms, with or without
4 # modification, are permitted provided that the following conditions are 4 # modification, are permitted provided that the following conditions are
5 # met: 5 # met:
6 # 6 #
7 # * Redistributions of source code must retain the above copyright 7 # * Redistributions of source code must retain the above copyright
8 # notice, this list of conditions and the following disclaimer. 8 # notice, this list of conditions and the following disclaimer.
9 # * Redistributions in binary form must reproduce the above 9 # * Redistributions in binary form must reproduce the above
10 # copyright notice, this list of conditions and the following disclaimer 10 # copyright notice, this list of conditions and the following disclaimer
(...skipping 20 matching lines...) Expand all
31 from webkitpy.layout_tests.models import test_failures 31 from webkitpy.layout_tests.models import test_failures
32 32
33 33
34 class TestResult(object): 34 class TestResult(object):
35 """Data object containing the results of a single test.""" 35 """Data object containing the results of a single test."""
36 36
37 @staticmethod 37 @staticmethod
38 def loads(string): 38 def loads(string):
39 return cPickle.loads(string) 39 return cPickle.loads(string)
40 40
41 def __init__(self, test_name, failures=None, test_run_time=None, has_stderr= False, reftest_type=None, pid=None, references=None, device_failed=False): 41 def __init__(self, test_name, failures=None, test_run_time=None, has_stderr= False, reftest_type=None, pid=None, references=None, device_failed=False, has_re paint_overlay=False):
42 self.test_name = test_name 42 self.test_name = test_name
43 self.failures = failures or [] 43 self.failures = failures or []
44 self.test_run_time = test_run_time or 0 # The time taken to execute the test itself. 44 self.test_run_time = test_run_time or 0 # The time taken to execute the test itself.
45 self.has_stderr = has_stderr 45 self.has_stderr = has_stderr
46 self.reftest_type = reftest_type or [] 46 self.reftest_type = reftest_type or []
47 self.pid = pid 47 self.pid = pid
48 self.references = references or [] 48 self.references = references or []
49 self.device_failed = device_failed 49 self.device_failed = device_failed
50 self.has_repaint_overlay = has_repaint_overlay
50 51
51 # FIXME: Setting this in the constructor makes this class hard to mutate . 52 # FIXME: Setting this in the constructor makes this class hard to mutate .
52 self.type = test_failures.determine_result_type(failures) 53 self.type = test_failures.determine_result_type(failures)
53 54
54 # These are set by the worker, not by the driver, so they are not passed to the constructor. 55 # These are set by the worker, not by the driver, so they are not passed to the constructor.
55 self.worker_name = '' 56 self.worker_name = ''
56 self.shard_name = '' 57 self.shard_name = ''
57 self.total_run_time = 0 # The time taken to run the test plus any refer ences, compute diffs, etc. 58 self.total_run_time = 0 # The time taken to run the test plus any refer ences, compute diffs, etc.
58 self.test_number = None 59 self.test_number = None
59 60
60 def __eq__(self, other): 61 def __eq__(self, other):
61 return (self.test_name == other.test_name and 62 return (self.test_name == other.test_name and
62 self.failures == other.failures and 63 self.failures == other.failures and
63 self.test_run_time == other.test_run_time) 64 self.test_run_time == other.test_run_time)
64 65
65 def __ne__(self, other): 66 def __ne__(self, other):
66 return not (self == other) 67 return not (self == other)
67 68
68 def has_failure_matching_types(self, *failure_classes): 69 def has_failure_matching_types(self, *failure_classes):
69 for failure in self.failures: 70 for failure in self.failures:
70 if type(failure) in failure_classes: 71 if type(failure) in failure_classes:
71 return True 72 return True
72 return False 73 return False
73 74
74 def dumps(self): 75 def dumps(self):
75 return cPickle.dumps(self) 76 return cPickle.dumps(self)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698