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

Unified Diff: chrome/browser/resources/media_internals/util.js

Issue 7972028: Display active media players on chrome://media-internals. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src
Patch Set: fixes Created 9 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: chrome/browser/resources/media_internals/util.js
diff --git a/chrome/browser/resources/media_internals/util.js b/chrome/browser/resources/media_internals/util.js
new file mode 100644
index 0000000000000000000000000000000000000000..7aca800a26fab4bb24ed754e87ada87dfcbfdeab
--- /dev/null
+++ b/chrome/browser/resources/media_internals/util.js
@@ -0,0 +1,92 @@
+// Copyright (c) 2011 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.
+
+cr.define('media', function() {
+ 'use strict';
+
+ /**
+ * The width and height of a bar drawn on a file canvas in pixels.
+ */
+ var BAR_WIDTH = 500;
+ var BAR_HEIGHT = 15;
+
+ // Maximum display length of a url in characters. Lengthy urls will be clipped
+ // and postfixed with an ellipsis.
+ var MAX_URL_LENGTH = 50;
+
+ /**
+ * Draws a 1px white horizontal line across |context|.
+ */
+ function drawLine(context, top) {
+ context.moveTo(0, top);
+ context.lineTo(BAR_WIDTH, top);
+ context.strokeStyle = '#fff';
+ context.stroke();
+ };
+
+ /**
+ * Creates an HTMLElement of type |type| with textContent |content|.
+ * @param {string} type The type of element to create.
+ * @param {string} content The content to place in the element.
+ * @return {HTMLElement} A newly initialized element.
+ */
+ function makeElement(type, content) {
+ var element = document.createElement(type);
+ element.textContent = content;
+ return element;
+ };
+
+ /**
+ * Creates a new <li> containing a <details> with a <summary> and sets
+ * properties to reference them.
+ * @return {Object} The new <li>.
+ */
+ function createDetailsLi() {
+ var li = document.createElement('li');
+ li.details = document.createElement('details');
+ li.summary = document.createElement('summary');
+ li.appendChild(li.details);
+ li.details.appendChild(li.summary);
+ return li
+ };
+
+ /**
+ * Prints each key-value pair in a dictionary to a row in a table.
+ * @param {Object} dict The dictionary to print.
+ * @param {HTMLElement} table The <table> element to print to.
+ */
+ function printDictionaryToTable(dict, table) {
+ table.textContent = '';
+ for (var key in dict) {
+ var tr = document.createElement('tr');
+ tr.appendChild(makeElement('td', key + ':'));
+ tr.appendChild(makeElement('td', dict[key]));
+ table.appendChild(tr);
+ }
+ return table;
+ };
+
+ /**
+ * Rounds a number, leaving the specified number of digits after the decimal,
+ * defaulting to 1.
+ * @param {number} n The number to be rounded.
+ * @param {int} digits The number of digits to leave after the decimal.
arv (Not doing code reviews) 2011/09/23 15:59:01 {number}
scherkus (not reviewing) 2011/09/23 17:43:23 Done.
+ * @return {number} |n| rounded to |digits| decimal places.
+ */
+ function round(n, digits) {
+ digits = digits || 1;
+ var factor = Math.pow(10, digits);
+ return Math.round(n * factor) / factor;
+ };
+
+ return {
+ BAR_WIDTH: BAR_WIDTH,
+ BAR_HEIGHT: BAR_HEIGHT,
+ drawLine: drawLine,
+ makeElement: makeElement,
+ createDetailsLi: createDetailsLi,
+ printDictionaryToTable: printDictionaryToTable,
+ round: round,
+ };
+});

Powered by Google App Engine
This is Rietveld 408576698