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

Unified Diff: content/browser/resources/media/new/player_info.js

Issue 23769009: Removes the --enable-new-media-internals flag. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed unused files from old-media-internals Created 7 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
« no previous file with comments | « content/browser/resources/media/new/media_internals.js ('k') | content/browser/resources/media/new/util.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/resources/media/new/player_info.js
diff --git a/content/browser/resources/media/new/player_info.js b/content/browser/resources/media/new/player_info.js
deleted file mode 100644
index af1f1944518f83ac0d811a5b1b1a84d931174e13..0000000000000000000000000000000000000000
--- a/content/browser/resources/media/new/player_info.js
+++ /dev/null
@@ -1,80 +0,0 @@
-// Copyright 2013 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.
-
-/**
- * @fileoverview A class for keeping track of the details of a player.
- */
-
-var PlayerInfo = (function() {
- 'use strict';
-
- /**
- * A class that keeps track of properties on a media player.
- * @param id A unique id that can be used to identify this player.
- */
- function PlayerInfo(id) {
- this.id = id;
- // The current value of the properties for this player.
- this.properties = {};
- // All of the past (and present) values of the properties.
- this.pastValues = {};
-
- // Every single event in the order in which they were received.
- this.allEvents = [];
- this.lastRendered = 0;
-
- this.firstTimestamp_ = -1;
- }
-
- PlayerInfo.prototype = {
- /**
- * Adds or set a property on this player.
- * This is the default logging method as it keeps track of old values.
- * @param timestamp The time in milliseconds since the Epoch.
- * @param key A String key that describes the property.
- * @param value The value of the property.
- */
- addProperty: function(timestamp, key, value) {
- // The first timestamp that we get will be recorded.
- // Then, all future timestamps are deltas of that.
- if (this.firstTimestamp_ === -1) {
- this.firstTimestamp_ = timestamp;
- }
-
- if (typeof key !== 'string') {
- throw new Error(typeof key + ' is not a valid key type');
- }
-
- this.properties[key] = value;
-
- if (!this.pastValues[key]) {
- this.pastValues[key] = [];
- }
-
- var recordValue = {
- time: timestamp - this.firstTimestamp_,
- key: key,
- value: value
- };
-
- this.pastValues[key].push(recordValue);
- this.allEvents.push(recordValue);
- },
-
- /**
- * Adds or set a property on this player.
- * Does not keep track of old values. This is better for
- * values that get spammed repeatedly.
- * @param timestamp The time in milliseconds since the Epoch.
- * @param key A String key that describes the property.
- * @param value The value of the property.
- */
- addPropertyNoRecord: function(timestamp, key, value) {
- this.addProperty(timestamp, key, value);
- this.allEvents.pop();
- }
- };
-
- return PlayerInfo;
-}());
« no previous file with comments | « content/browser/resources/media/new/media_internals.js ('k') | content/browser/resources/media/new/util.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698