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

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

Issue 134243003: Create separate telemetry media-perf ChromeOS benchmarks for 4k and non-4k content. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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 | tools/perf/page_sets/tough_video_cases.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/benchmarks/media.py
diff --git a/tools/perf/benchmarks/media.py b/tools/perf/benchmarks/media.py
index bbcdef5aba2486d9c0a61dccf11e8accaed75e6b..61f11dfa219edd100a9ce86a8604facc4c3884e7 100644
--- a/tools/perf/benchmarks/media.py
+++ b/tools/perf/benchmarks/media.py
@@ -41,14 +41,27 @@ class MediaAndroid(test.Test):
'page_filter_exclude': '.*(crowd|garden).*'
shadi1 2014/01/31 17:50:39 Can we update this to use labels for consistency?
anandc 2014/01/31 17:58:47 Done.
}
+class MediaChromeOSLowEnd(test.Test):
+ """Obtains media metrics for key user scenarios on less powerful
+ ChromeOS platforms."""
+ test = media.Media
+ tag = 'chromeOSLowEnd'
+ page_set = 'page_sets/tough_video_cases.json'
+ # Exclude 4K and 50fps test files: crbug/331816
+ options = {
+ 'page_label_filter_exclude': '4k,50fps'
+ }
+
class MediaChromeOS(test.Test):
- """Obtains media metrics for key user scenarios on ChromeOS."""
+ """Obtains media metrics for key user scenarios on ChromeOS platforms that
+ support HWDecode or are able to play hd/high-res content even with
+ software-decoding."""
test = media.Media
tag = 'chromeOS'
page_set = 'page_sets/tough_video_cases.json'
- # Exclude crowd* media files (50fps 2160p): crbug/331816
+ # Exclude 50fps test files: crbug/331816
options = {
- 'page_filter_exclude': '.*crowd.*'
+ 'page_label_filter_exclude': '50fps'
}
class MediaSourceExtensions(test.Test):
« no previous file with comments | « no previous file | tools/perf/page_sets/tough_video_cases.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698