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

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

Issue 2860753002: Remove disabling of reference builds on V8 benchmarks (Closed)
Patch Set: Created 3 years, 8 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/v8_browsing.py
diff --git a/tools/perf/benchmarks/v8_browsing.py b/tools/perf/benchmarks/v8_browsing.py
index bcd8b7e1f40cdc15286e20724411ac914efc0864..5613c74d880277a9dfe8a4e9990ffaf6eb2fa9e0 100644
--- a/tools/perf/benchmarks/v8_browsing.py
+++ b/tools/perf/benchmarks/v8_browsing.py
@@ -154,7 +154,6 @@ class V8DesktopBrowsingBenchmark(_V8BrowsingBenchmark):
@benchmark.Owner(emails=['ulan@chromium.org'])
@benchmark.Enabled('android')
-@benchmark.Disabled('reference') # http://crbug.com/628631
class V8MobileBrowsingBenchmark(_V8BrowsingBenchmark):
PLATFORM = 'mobile'
@@ -163,7 +162,6 @@ class V8MobileBrowsingBenchmark(_V8BrowsingBenchmark):
return 'v8.browsing_mobile'
-@benchmark.Disabled('reference') # http://crbug.com/700390
@benchmark.Disabled('android')
@benchmark.Disabled('all')
@benchmark.Owner(emails=['mvstaton@chromium.org'])
@@ -181,7 +179,6 @@ class V8DesktopTurboBrowsingBenchmark(_V8BrowsingBenchmark):
-@benchmark.Disabled('reference') # http://crbug.com/628631
@benchmark.Enabled('android')
@benchmark.Disabled('all')
@benchmark.Owner(emails=['mvstaton@chromium.org'])
@@ -198,7 +195,6 @@ class V8MobileTurboBrowsingBenchmark(_V8BrowsingBenchmark):
return 'v8.browsing_mobile_turbo'
-@benchmark.Disabled('reference') # http://crbug.com/700390
@benchmark.Disabled('android')
@benchmark.Owner(emails=['hablich@chromium.org'])
class V8DesktopClassicBrowsingBenchmark(_V8BrowsingBenchmark):
@@ -214,7 +210,6 @@ class V8DesktopClassicBrowsingBenchmark(_V8BrowsingBenchmark):
return 'v8.browsing_desktop_classic'
-@benchmark.Disabled('reference') # http://crbug.com/628631
@benchmark.Enabled('android')
@benchmark.Owner(emails=['hablich@chromium.org'])
class V8MobileClassicBrowsingBenchmark(_V8BrowsingBenchmark):
@@ -242,7 +237,6 @@ class V8RuntimeStatsDesktopBrowsingBenchmark(
return 'v8.runtimestats.browsing_desktop'
-@benchmark.Disabled('reference') # http://crbug.com/700390
@benchmark.Disabled('android')
@benchmark.Disabled('all')
@benchmark.Owner(emails=['mythria@chromium.org'])
@@ -260,8 +254,7 @@ class V8RuntimeStatsDesktopTurboBrowsingBenchmark(
return 'v8.runtimestats.browsing_desktop_turbo'
-@benchmark.Disabled('reference', # http://crbug.com/700390
- 'win') # http://crbug.com/704197
+@benchmark.Disabled('win') # http://crbug.com/704197
@benchmark.Disabled('android')
@benchmark.Owner(emails=['hablich@chromium.org'])
class V8RuntimeStatsDesktopClassicBrowsingBenchmark(
« 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