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

Unified Diff: perf_insights/perf_insights/map_single_trace.html

Issue 1956073002: Removed the isTimeHighResolution check from mapSingleTrace. It serves no purpose anymore. (Closed) Base URL: https://github.com/catapult-project/catapult.git@master
Patch Set: Created 4 years, 7 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: perf_insights/perf_insights/map_single_trace.html
diff --git a/perf_insights/perf_insights/map_single_trace.html b/perf_insights/perf_insights/map_single_trace.html
index 583aae103776f43a25eeb4b13bfbdc78008c050b..90b90917782bc2ad2e29dec473d86ec3c5614532 100644
--- a/perf_insights/perf_insights/map_single_trace.html
+++ b/perf_insights/perf_insights/map_single_trace.html
@@ -34,14 +34,6 @@ tr.exportTo('pi', function() {
}
function mapSingleTrace(result, model, options, mapFunction) {
- // Do not map the trace if its timer is low resolution.
- if (!model.isTimeHighResolution) {
- var err = new Error('Trace doesn\'t have high resolution time, ' +
- 'cannot map.');
- err.name = 'LowResolutionTimeError';
- throw err;
- }
-
// Map the function.
var numPairsBeforeMapping = tr.b.dictionaryLength(result.pairs);
var numFailuresBeforeMapping = result.failures.length;
« 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