Adds cache and buffer graphs to the properties pane.

content/browser/resources/media/new/client_renderer.js

 // 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.
 
 var ClientRenderer = (function() {
   var ClientRenderer = function() {
     this.playerListElement = document.getElementById('player-list');
     this.audioStreamListElement = document.getElementById('audio-stream-list');
     this.propertiesTable = document.getElementById('property-table');
     this.logTable = document.getElementById('log');
+    this.graphElement = document.getElementById('graphs');
 
     this.selectedPlayer = null;
     this.selectedStream = null;
 
     this.selectedPlayerLogIndex = 0;
+
+    this.bufferCanvas = document.createElement('canvas');
+    this.bufferCanvas.width = media.BAR_WIDTH;
+    this.bufferCanvas.height = media.BAR_HEIGHT;
   };
 
   function removeChildren(element) {
     while (element.hasChildNodes()) {
       element.removeChild(element.lastChild);
     }
   };
 
   function createButton(text, select_cb) {
     var button = document.createElement('button');
@@ skipping 43 matching lines @@
      * Called when a property on a player is changed.
      * @param players The entire map of id -> player.
      * @param player The player that had its property changed.
      * @param key The name of the property that was changed.
      * @param value The new value of the property.
      */
     playerUpdated: function(players, player, key, value) {
       if (player === this.selectedPlayer) {
         this.drawProperties_(player.properties);
         this.drawLog_();
+        this.drawGraphs_();
       }
       if (key === 'name' || key === 'url') {
         this.redrawPlayerList_(players);
       }
     },
 
     redrawAudioStreamList_: function(streams) {
       removeChildren(this.audioStreamListElement);
 
       for (id in streams) {
         var li = document.createElement('li');
         li.appendChild(createButton(
             id, this.selectAudioStream_.bind(this, streams[id])));
         this.audioStreamListElement.appendChild(li);
       }
     },
 
     selectAudioStream_: function(audioStream) {
       this.selectedStream = audioStream;
       this.selectedPlayer = null;
       this.drawProperties_(audioStream);
       removeChildren(this.logTable.querySelector('tbody'));
+      removeChildren(this.graphElement);
     },
 
     redrawPlayerList_: function(players) {
       removeChildren(this.playerListElement);
 
       for (id in players) {
         var li = document.createElement('li');
         var player = players[id];
         var usableName = player.properties.name ||
             player.properties.url ||
             'player ' + player.id;
 
         li.appendChild(createButton(
             usableName, this.selectPlayer_.bind(this, player)));
         this.playerListElement.appendChild(li);
       }
     },
 
     selectPlayer_: function(player) {
       this.selectedPlayer = player;
       this.selectedPlayerLogIndex = 0;
       this.selectedStream = null;
       this.drawProperties_(player.properties);
 
       removeChildren(this.logTable.querySelector('tbody'));
+      removeChildren(this.graphElement);
       this.drawLog_();
+      this.drawGraphs_();
     },
 
     drawProperties_: function(propertyMap) {
       removeChildren(this.propertiesTable);
 
       for (key in propertyMap) {
         var value = propertyMap[key];
+
         var row = this.propertiesTable.insertRow(-1);
         var keyCell = row.insertCell(-1);
         var valueCell = row.insertCell(-1);
 
         keyCell.appendChild(document.createTextNode(key));
         valueCell.appendChild(document.createTextNode(value));
       }
     },
 
     appendEventToLog_: function(event) {
       var row = this.logTable.querySelector('tbody').insertRow(-1);
 
       row.insertCell(-1).appendChild(document.createTextNode(
           util.millisecondsToString(event.time)));
       row.insertCell(-1).appendChild(document.createTextNode(event.key));
       row.insertCell(-1).appendChild(document.createTextNode(event.value));
     },
 
     drawLog_: function() {
       var toDraw = this.selectedPlayer.allEvents.slice(
           this.selectedPlayerLogIndex);
       toDraw.forEach(this.appendEventToLog_.bind(this));
       this.selectedPlayerLogIndex = this.selectedPlayer.allEvents.length;
+    },
+
+    drawGraphs_: function() {
+      removeChildren(this.graphElement);

[scherkus (not reviewing), 2013/09/06 18:16:47]

won't this cause line 209 to always evaluate to true?

[Ty Overby, 2013/09/06 19:56:37]

Done.

+
+      var addToGraphs = (function(name, graph) {

[scherkus (not reviewing), 2013/09/06 18:16:47]

considering you're already passing in this. member variables, how about dropping
bind() in favour of something more straightforward:

function addGraph(graphElement, name, graph) {
  // ...
  graphElement.appendChild(li);
}

addGraph(this.graphElement, 'buffered', this.bufferedCanvas);

[Ty Overby, 2013/09/06 19:56:37]

Done.

+          var li = document.createElement('li');
+          li.appendChild(graph);
+          li.appendChild(document.createTextNode(name));
+          this.graphElement.appendChild(li);
+      }).bind(this);
+
+      var url = this.selectedPlayer.properties.url;
+      if (!url) {
+        return;
+      }
+
+      var cache = media.cacheForUrl(url);
+
+      var player = this.selectedPlayer;
+      var props = player.properties;
+
+      var cacheExists = false,
+          bufferExists = false;

[scherkus (not reviewing), 2013/09/06 18:16:47]

might as well use separate var statement here

[Ty Overby, 2013/09/06 19:56:37]

Done.

+
+      if (props['buffer_start'] !== undefined &&
+          props['buffer_current'] !== undefined &&
+          props['buffer_end'] !== undefined &&
+          props['total_bytes'] !== undefined) {
+        this.drawBufferGraph_(props['buffer_start'],
+                              props['buffer_current'],
+                              props['buffer_end'],
+                              props['total_bytes']);
+        bufferExists = true;
+      }
+
+      if (cache) {
+        cache.size = cache.size || +player.properties['total_bytes'];

[scherkus (not reviewing), 2013/09/06 18:16:47]

can you rewrite this to avoid type trickery

[Ty Overby, 2013/09/06 19:56:37]

Done.

+        cache.generateDetails();
+        cacheExists = true;
+      }
+
+      if (!this.graphElement.hasChildNodes()) {
+        if (bufferExists) {
+          addToGraphs('buffer', this.bufferCanvas);
+        }
+        if (cacheExists) {
+          addToGraphs('cache read', cache.readCanvas);
+          addToGraphs('cache write', cache.writeCanvas);
+        }
+      }
+    },
+
+    drawBufferGraph_: function(start, current, end, size) {
+      var ctx = this.bufferCanvas.getContext('2d');
+      var width = this.bufferCanvas.width;
+      var height = this.bufferCanvas.height;
+      ctx.fillStyle = '#aaa';
+      ctx.fillRect(0, 0, width, height);
+
+      var scale_factor = width / size;
+      var left = start * scale_factor;
+      var middle = current * scale_factor;
+      var right = end * scale_factor;
+
+      ctx.fillStyle = '#a0a';
+      ctx.fillRect(left, 0, middle - left, height);
+      ctx.fillStyle = '#aa0';
+      ctx.fillRect(middle, 0, right - middle, height);
     }
   };
 
   return ClientRenderer;
 })();