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

Unified Diff: ui/file_manager/video_player/js/video_player_metrics.js

Issue 760853003: Adds histograms for casting feature of Video Player (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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: ui/file_manager/video_player/js/video_player_metrics.js
diff --git a/ui/file_manager/file_manager/foreground/js/metrics.js b/ui/file_manager/video_player/js/video_player_metrics.js
similarity index 54%
copy from ui/file_manager/file_manager/foreground/js/metrics.js
copy to ui/file_manager/video_player/js/video_player_metrics.js
index cae363523bdc6fe9e0d3d393485dbc0c5f57bf2a..4a17c42302ab9ce160e7fce4827de52aecdd812d 100644
--- a/ui/file_manager/file_manager/foreground/js/metrics.js
+++ b/ui/file_manager/video_player/js/video_player_metrics.js
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2014 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.
@@ -8,10 +8,23 @@
* To be included as a first script in main.html
*/
+/**
+ * @extends {metricsBase}
+ */
var metrics = metricsBase;
-metrics.startInterval('Load.Total');
-metrics.startInterval('Load.Script');
+/**
+ * Values for "VideoPlayer.CastExtensionStaus" metrics.
+ * @enum {number}
+ */
+metrics.CAST_EXTENSION = {
+ EXTENSION_UNAVAILABLE: 0,
+ API_INSTALLATION_FAILED: 1,
+ API_LOAD_FAILED: 2,
+ API_INSTALLED_AND_LOADED: 3,
+ API_AVAILABLE_AND_LOADED: 4,
+ MAX_VALUE: 4
+};
/**
* Convert a short metric name to the full format.
@@ -22,5 +35,5 @@ metrics.startInterval('Load.Script');
* @private
*/
metrics.convertName_ = function(name) {
- return 'FileBrowser.' + name;
+ return 'VideoPlayer.' + name;
};

Powered by Google App Engine
This is Rietveld 408576698