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

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

Issue 2874033005: Move memory.long_running_dual_browser_test benchmark to tools/perf/contrib/ (Closed)
Patch Set: Created 3 years, 7 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 | « tools/perf/benchmark.csv ('k') | tools/perf/contrib/memory_long_running/OWNERS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/benchmarks/memory.py
diff --git a/tools/perf/benchmarks/memory.py b/tools/perf/benchmarks/memory.py
index d5b2122c9ae2c30e43971a3f5411a6eb931a42b0..8d3d76c1db1528c809b0bf1773a55c84a3ccb080 100644
--- a/tools/perf/benchmarks/memory.py
+++ b/tools/perf/benchmarks/memory.py
@@ -150,37 +150,6 @@ class DualBrowserBenchmark(_MemoryInfra):
return not _IGNORED_STATS_RE.search(value.name)
-# Benchmark disabled by default. Force to run with --also-run-disabled-tests.
-@benchmark.Disabled('all')
-@benchmark.Owner(emails=['perezju@chromium.org'])
-class LongRunningDualBrowserBenchmark(_MemoryInfra):
- """Measures memory during prolonged usage of alternating browsers.
-
- Same as memory.dual_browser_test, but the test is run for 60 iterations
- and the browser is *not* restarted between page set repeats.
- """
- page_set = page_sets.DualBrowserStorySet
- options = {'pageset_repeat': 60}
-
- @classmethod
- def Name(cls):
- return 'memory.long_running_dual_browser_test'
-
- @classmethod
- def ShouldTearDownStateAfterEachStoryRun(cls):
- return False
-
- @classmethod
- def ShouldTearDownStateAfterEachStorySetRun(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.
- return not _IGNORED_STATS_RE.search(value.name)
-
-
@benchmark.Enabled('android') # catapult:#3176
@benchmark.Owner(emails=['bashi@chromium.org'])
class RendererMemoryBlinkMemoryMobile(_MemoryInfra):
« no previous file with comments | « tools/perf/benchmark.csv ('k') | tools/perf/contrib/memory_long_running/OWNERS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698