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

Unified Diff: tracing/tracing/metrics/vr/webvr_metric_test.html

Issue 3001843002: Add TBMv2 metrics for WebVR. (Closed)
Patch Set: Addressed Ben's comments Created 3 years, 4 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 | « tracing/tracing/metrics/vr/webvr_metric.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/metrics/vr/webvr_metric_test.html
diff --git a/tracing/tracing/metrics/vr/webvr_metric_test.html b/tracing/tracing/metrics/vr/webvr_metric_test.html
new file mode 100644
index 0000000000000000000000000000000000000000..823d414a19e78d6afcc6da792b1f6ebd2c68c44d
--- /dev/null
+++ b/tracing/tracing/metrics/vr/webvr_metric_test.html
@@ -0,0 +1,65 @@
+<!DOCTYPE html>
+<!--
+Copyright 2017 The Chromium Authors. All rights reserved.
+Use of this source code is governed by a BSD-style license that can be
+found in the LICENSE file.
+-->
+
+<link rel="import" href="/tracing/core/test_utils.html">
+<link rel="import" href="/tracing/metrics/vr/webvr_metric.html">
+<link rel="import" href="/tracing/model/counter.html">
+<link rel="import" href="/tracing/model/model.html">
+<link rel="import" href="/tracing/value/histogram_set.html">
+
+<script>
+'use strict';
+
+tr.b.unittest.testSuite(function() {
+ function createModel() {
+ const model = tr.c.TestUtils.newModel(
+ function(model) {
+ const process = model.getOrCreateProcess(1);
+ const fpsCounter = process.getOrCreateCounter('gpu', 'WebVR FPS');
+ const fpsSeries = new tr.model.CounterSeries('value', 0);
+ fpsSeries.addCounterSample(1, 59);
+ fpsSeries.addCounterSample(3, 60);
+ fpsCounter.addSeries(fpsSeries);
+
+ const frameTimeCounter = process.getOrCreateCounter(
+ 'gpu', 'WebVR frame time (ms)');
+ const renderingSeries = new tr.model.CounterSeries('rendering', 0);
+ renderingSeries.addCounterSample(1, 3);
+ renderingSeries.addCounterSample(2, 4);
+ frameTimeCounter.addSeries(renderingSeries);
+ const javascriptSeries = new tr.model.CounterSeries('javascript', 0);
+ javascriptSeries.addCounterSample(1, 5);
+ javascriptSeries.addCounterSample(2, 6);
+ frameTimeCounter.addSeries(javascriptSeries);
+ });
+ return model;
+ }
+
+ test('webvrMetric', function() {
+ const histograms = new tr.v.HistogramSet();
+ const model = createModel();
+ tr.metrics.vr.webvrMetric(histograms, model);
+
+ const fpsValue = histograms.getHistogramNamed('webvr_fps');
+ assert.strictEqual(fpsValue.max, 60);
+ assert.strictEqual(fpsValue.min, 59);
+ assert.strictEqual(fpsValue.average, 59.5);
+
+ const renderingValue = histograms.getHistogramNamed(
+ 'webvr_frame_time_rendering');
+ assert.strictEqual(renderingValue.max, 4);
+ assert.strictEqual(renderingValue.min, 3);
+ assert.strictEqual(renderingValue.average, 3.5);
+
+ const javascriptValue = histograms.getHistogramNamed(
+ 'webvr_frame_time_javascript');
+ assert.strictEqual(javascriptValue.max, 6);
+ assert.strictEqual(javascriptValue.min, 5);
+ assert.strictEqual(javascriptValue.average, 5.5);
+ });
+});
+</script>
« no previous file with comments | « tracing/tracing/metrics/vr/webvr_metric.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698