Chromium Code Reviews| Index: tools/perf/benchmarks/memory_infra.py |
| diff --git a/tools/perf/benchmarks/memory_infra.py b/tools/perf/benchmarks/memory_infra.py |
| index ce8f3516e0280851509f93c9741c528a6f157c8d..f79f1007b7bcfa4fb3695506391187b983da46c6 100644 |
| --- a/tools/perf/benchmarks/memory_infra.py |
| +++ b/tools/perf/benchmarks/memory_infra.py |
| @@ -62,6 +62,10 @@ class MemoryBenchmarkTop10Mobile(_MemoryInfra): |
| return 'memory.top_10_mobile' |
| @classmethod |
| + def ShouldTearDownStateAfterEachStoryRun(cls): |
| + return False |
|
perezju
2016/09/02 09:31:36
I've just realized we cannot tear down the state o
|
| + |
| + @classmethod |
| def ShouldDisable(cls, possible_browser): |
| # TODO(crbug.com/586148): Benchmark should not depend on DeskClock app. |
| return not possible_browser.platform.CanLaunchApplication( |
| @@ -87,6 +91,10 @@ class MemoryBenchmarkTop10MobileStress(MemoryBenchmarkTop10Mobile): |
| return 'memory.top_10_mobile_stress' |
| @classmethod |
| + def ShouldTearDownStateAfterEachStoryRun(cls): |
| + return False |
|
petrcermak
2016/09/02 09:40:55
You don't need this here because the class inherit
perezju
2016/09/02 10:53:37
Good catch.
|
| + |
| + @classmethod |
| def ShouldTearDownStateAfterEachStorySetRun(cls): |
| return False |
| @@ -108,6 +116,10 @@ class DualBrowserBenchmark(_MemoryInfra): |
| return 'memory.dual_browser_test' |
| @classmethod |
| + def ShouldTearDownStateAfterEachStoryRun(cls): |
| + return False |
| + |
| + @classmethod |
| def ValueCanBeAddedPredicate(cls, value, is_first_result): |
| # TODO(crbug.com/610962): Remove this stopgap when the perf dashboard |
| # is able to cope with the data load generated by TBMv2 metrics. |
| @@ -130,6 +142,10 @@ class LongRunningDualBrowserBenchmark(_MemoryInfra): |
| return 'memory.long_running_dual_browser_test' |
| @classmethod |
| + def ShouldTearDownStateAfterEachStoryRun(cls): |
| + return False |
| + |
| + @classmethod |
| def ShouldTearDownStateAfterEachStorySetRun(cls): |
| return False |