Index: tools/telemetry/telemetry/page/page_test.py |
diff --git a/tools/telemetry/telemetry/page/page_test.py b/tools/telemetry/telemetry/page/page_test.py |
index 74bab7dac8271dc23947c7ec1486f19b663b3952..386f68d602030733ccda77b0e5687b1a5d02ae64 100644 |
--- a/tools/telemetry/telemetry/page/page_test.py |
+++ b/tools/telemetry/telemetry/page/page_test.py |
@@ -54,9 +54,6 @@ class PageTest(command_line.Command): |
subclasses to run. |
discard_first_run: Discard the first run of this page. This is |
usually used with page_repeat and pageset_repeat options. |
- attempts: The number of attempts to run if we encountered |
- infrastructure problems (as opposed to test issues), such as |
- losing a browser. |
max_failures: The number of page failures allowed before we stop |
running other pages. |
is_action_name_to_run_optional: Determines what to do if |
@@ -73,7 +70,6 @@ class PageTest(command_line.Command): |
needs_browser_restart_after_each_page=False, |
discard_first_result=False, |
clear_cache_before_each_run=False, |
- attempts=3, |
max_failures=None, |
is_action_name_to_run_optional=False): |
super(PageTest, self).__init__() |
@@ -90,10 +86,8 @@ class PageTest(command_line.Command): |
self._discard_first_result = discard_first_result |
self._clear_cache_before_each_run = clear_cache_before_each_run |
self._close_tabs_before_run = True |
- self._attempts = attempts |
self._max_failures = max_failures |
self._is_action_name_to_run_optional = is_action_name_to_run_optional |
- assert self._attempts > 0, 'Test attempts must be greater than 0' |
# If the test overrides the TabForPage method, it is considered a multi-tab |
# test. The main difference between this and a single-tab test is that we |
# do not attempt recovery for the former if a tab or the browser crashes, |
@@ -138,12 +132,8 @@ class PageTest(command_line.Command): |
@property |
def attempts(self): |
"""Maximum number of times test will be attempted.""" |
- return self._attempts |
- |
- @attempts.setter |
- def attempts(self, count): |
- assert self._attempts > 0, 'Test attempts must be greater than 0' |
- self._attempts = count |
+ # Do NOT override this method (crbug.com/422339). |
+ return 3 |
@property |
def max_failures(self): |