Fix MediaSource renderer to limit memory consumption.

remoting/webapp/client_plugin.js

 // Copyright (c) 2012 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
  * Class that wraps low-level details of interacting with the client plugin.
  *
  * This abstracts a <embed> element and controls the plugin which does
  * the actual remoting work. It also handles differences between
@@ skipping 309 matching lines @@
 
   } else if (message.method == 'mediaSourceData') {
     if (!(message.data['buffer'] instanceof ArrayBuffer)) {
       console.error('Invalid mediaSourceData message:', message.data);
       return;
     }
     if (!this.mediaSourceRenderer_) {
       console.error('Unexpected mediaSourceData.');
       return;
     }
+    // keyframe flag may be absent from the message.
+    var keyframe = !!message.data['keyframe'];

[Jamie, 2014/06/11 21:41:42]

Does this pass jscompile? I'm surprised you don't need to include a type.

[Sergey Ulanov, 2014/06/11 23:41:12]

Yes it does.

     this.mediaSourceRenderer_.onIncomingData(
-        (/** @type {ArrayBuffer} */ message.data['buffer']));
+        (/** @type {ArrayBuffer} */ message.data['buffer']), keyframe);
   }
 };
 
 /**
  * Deletes the plugin.
  */
 remoting.ClientPlugin.prototype.cleanup = function() {
   this.plugin.parentNode.removeChild(this.plugin);
 };
 
@@ skipping 359 matching lines @@
     this.plugin.style.width = width + 'px';
     this.plugin.style.height = height + 'px';
     // Center the plugin just underneath the "Connnecting..." dialog.
     var parentNode = this.plugin.parentNode;
     var dialog = document.getElementById('client-dialog');
     var dialogRect = dialog.getBoundingClientRect();
     parentNode.style.top = (dialogRect.bottom + 16) + 'px';
     parentNode.style.left = (window.innerWidth - width) / 2 + 'px';
   }
 };