Display active media players on chrome://media-internals.

chrome/browser/resources/media_internals/util.js

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..bb31599889e107eaae604a8a3c80402b5ac9e39e
--- /dev/null
+++ b/chrome/browser/resources/media_internals/util.js
@@ -0,0 +1,88 @@
+// 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;
+
+  /**
+   * 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) {

[arv (Not doing code reviews), 2011/10/18 20:50:07]

Please rename. print is not the right word here

[scherkus (not reviewing), 2011/10/19 00:19:43]

s/print/append

+    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 {number} digits The number of digits to leave after the decimal.
+   * @return {number} |n| rounded to |digits| decimal places.
+   */
+  function round(n, digits) {

[arv (Not doing code reviews), 2011/10/18 20:50:07]

Can you use toFixed instead?

[scherkus (not reviewing), 2011/10/19 00:19:43]

Done.

+    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,
+  };
+});