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

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

Issue 2742293002: WebRTC: Add a TBMv2 version of the webrtc.webrtc_smoothness benchmark. (Closed)
Patch Set: Address nits. Created 3 years, 9 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 | « testing/buildbot/chromium.perf.fyi.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/benchmarks/webrtc.py
diff --git a/tools/perf/benchmarks/webrtc.py b/tools/perf/benchmarks/webrtc.py
index bf9582ab19f1b16890fa26d71c138661510c732f..947ab94aade30e83e7860715df3e065793aea448 100644
--- a/tools/perf/benchmarks/webrtc.py
+++ b/tools/perf/benchmarks/webrtc.py
@@ -94,3 +94,34 @@ class WebrtcRendering(perf_benchmark.PerfBenchmark):
@classmethod
def Name(cls):
return 'webrtc.webrtc_smoothness'
+
+
+# WebrtcRenderingTBMv2 must be a PerfBenchmark, and not a _Webrtc, because it is
+# a timeline-based metric.
+print dir(benchmark)
+@benchmark.Owner(emails=['ehmaldonado@chromium.org',
+ 'phoglund@chromium.org',
+ 'qiangchen@chromium.org'])
+class WebrtcRenderingTBMv2(perf_benchmark.PerfBenchmark):
+ """Specific time measurements (e.g. fps, smoothness) for WebRtc rendering."""
+
+ page_set = page_sets.WebrtcRenderingPageSet
+
+ def CreateTimelineBasedMeasurementOptions(self):
+ category_filter = chrome_trace_category_filter.ChromeTraceCategoryFilter(
+ filter_string='webrtc,toplevel')
+ options = timeline_based_measurement.Options(category_filter)
+ options.SetTimelineBasedMetrics([
+ 'cpuTimeMetric',
+ 'expectedQueueingTimeMetric',
+ 'webrtcRenderingMetric',
+ ])
+ return options
+
+ def SetExtraBrowserOptions(self, options):
+ options.AppendExtraBrowserArgs('--use-fake-device-for-media-stream')
+ options.AppendExtraBrowserArgs('--use-fake-ui-for-media-stream')
+
+ @classmethod
+ def Name(cls):
+ return 'webrtc.webrtc_smoothness_tbmv2'
« no previous file with comments | « testing/buildbot/chromium.perf.fyi.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698