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

Unified Diff: tools/perf/benchmarks/memory_infra.py

Issue 2301383002: [tools/perf] Explicitly disable state tear down on some benchmarks (Closed)
Patch Set: petr's comment Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..1e03843c3aa8e3e270c87883f8ece7c231a1395b 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
+
+ @classmethod
def ShouldDisable(cls, possible_browser):
# TODO(crbug.com/586148): Benchmark should not depend on DeskClock app.
return not possible_browser.platform.CanLaunchApplication(
@@ -108,6 +112,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 +138,10 @@ class LongRunningDualBrowserBenchmark(_MemoryInfra):
return 'memory.long_running_dual_browser_test'
@classmethod
+ def ShouldTearDownStateAfterEachStoryRun(cls):
+ return False
+
+ @classmethod
def ShouldTearDownStateAfterEachStorySetRun(cls):
return False
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698