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

Unified Diff: tracing/tracing/value/diagnostics/related_histogram_breakdown_test.html

Issue 3009553002: Refactor Histogram relationship diagnostics. (Closed)
Patch Set: Created 3 years, 3 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
Index: tracing/tracing/value/diagnostics/related_histogram_breakdown_test.html
diff --git a/tracing/tracing/value/diagnostics/related_histogram_breakdown_test.html b/tracing/tracing/value/diagnostics/related_histogram_breakdown_test.html
deleted file mode 100644
index c4d7729d4ebb46b05b775dd4aebdc7babd59bffb..0000000000000000000000000000000000000000
--- a/tracing/tracing/value/diagnostics/related_histogram_breakdown_test.html
+++ /dev/null
@@ -1,83 +0,0 @@
-<!DOCTYPE html>
-<!--
-Copyright 2016 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/extras/chrome/chrome_user_friendly_category_driver.html">
-<link rel="import" href="/tracing/model/event_set.html">
-<link rel="import" href="/tracing/value/diagnostics/related_histogram_breakdown.html">
-<link rel="import" href="/tracing/value/histogram.html">
-<link rel="import" href="/tracing/value/histogram_set.html">
-
-<script>
-'use strict';
-
-tr.b.unittest.testSuite(function() {
- const TEST_BOUNDARIES = tr.v.HistogramBinBoundaries.createLinear(0, 10, 10);
-
- test('build', function() {
- const histograms = new tr.v.HistogramSet();
- let events = new tr.model.EventSet([
- {
- guid: 9,
- title: 'V8.Execute',
- cpuSelfTime: 1,
- stableId: '11.11'
- },
- {
- guid: 10,
- title: 'V8.Execute',
- cpuSelfTime: 2,
- stableId: '22.22'
- },
- {
- guid: 11,
- title: 'UpdateLayoutTree',
- cpuSelfTime: 3,
- stableId: '33.33'
- },
- {
- guid: 12,
- title: 'UpdateLayoutTree',
- cpuSelfTime: undefined,
- stableId: '44.44'
- }
- ]);
-
- const sampleForEvent = undefined;
- const diagnostic = tr.v.d.RelatedHistogramBreakdown.buildFromEvents(
- histograms, 'foo-', events,
- tr.e.chrome.ChromeUserFriendlyCategoryDriver.fromEvent,
- tr.b.Unit.byName.timeDurationInMs, sampleForEvent, TEST_BOUNDARIES);
-
- assert.lengthOf(diagnostic, 2);
- const scriptHist = histograms.getHistogramNamed('foo-script_execute');
- assert.instanceOf(scriptHist, tr.v.Histogram);
- assert.lengthOf(scriptHist.allBins[2].diagnosticMaps, 1);
- assert.strictEqual(scriptHist, diagnostic.get('script_execute'));
- events = [...events];
- let relatedEvents = scriptHist.allBins[2].diagnosticMaps[0]
- .get('relatedEvents');
- assert.lengthOf(relatedEvents, 1);
- assert.strictEqual(events[0], [...relatedEvents][0]);
- assert.lengthOf(scriptHist.allBins[3].diagnosticMaps, 1);
- assert.lengthOf(
- scriptHist.allBins[3].diagnosticMaps[0]
- .get('relatedEvents'), 1);
- assert.strictEqual(events[1],
- [...scriptHist.allBins[3].diagnosticMaps[0]
- .get('relatedEvents')][0]);
-
- const styleHist = histograms.getHistogramNamed('foo-style');
- assert.instanceOf(styleHist, tr.v.Histogram);
- assert.lengthOf(styleHist.allBins[4].diagnosticMaps, 1);
- relatedEvents = styleHist.allBins[4].diagnosticMaps[0]
- .get('relatedEvents');
- assert.strictEqual(styleHist, diagnostic.get('style'));
- assert.lengthOf(relatedEvents, 1);
- assert.strictEqual(events[2], [...relatedEvents][0]);
- });
-});
-</script>

Powered by Google App Engine
This is Rietveld 408576698