Add getUserMedia info to the dump and add browser tests.

Add getUserMedia info to the dump and add browser tests.

BUG=304023
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244281

[jiayl, 2014-01-08 23:39:55]

This will help debugging getUserMedia issues. PTAL. Thanks!

[vrk (LEFT CHROMIUM), 2014-01-09 00:26:30]

lgtm

https://codereview.chromium.org/129833002/diff/1/content/browser/resources/me...
File content/browser/resources/media/webrtc_internals.js (right):

https://codereview.chromium.org/129833002/diff/1/content/browser/resources/me...
content/browser/resources/media/webrtc_internals.js:256: // TODO(jiayl): add the
getUserMedia info to the tabbed UI.
nit: remove TODO?

https://codereview.chromium.org/129833002/diff/1/content/browser/resources/me...
content/browser/resources/media/webrtc_internals.js:267: // TODO(jiayl): remove
the getUserMedia info from the tabbed UI.
nit: remove TODO?

https://codereview.chromium.org/129833002/diff/1/content/browser/resources/me...
content/browser/resources/media/webrtc_internals.js:269: if
(userMediaRequests[i].rid == data.rid) {
nit: no {}

[commit-bot: I haz the power, 2014-01-10 19:21:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jiayl@chromium.org/129833002/60001

[commit-bot: I haz the power, 2014-01-10 19:21:54]

Failed to apply patch for content/browser/resources/media/dump_creator.js:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file content/browser/resources/media/dump_creator.js
  Hunk #1 FAILED at 89.
  1 out of 1 hunk FAILED -- saving rejects to file
content/browser/resources/media/dump_creator.js.rej

Patch:       content/browser/resources/media/dump_creator.js
Index: content/browser/resources/media/dump_creator.js
diff --git a/content/browser/resources/media/dump_creator.js
b/content/browser/resources/media/dump_creator.js
index
91b02205fc5526bc3787cb3c0b78148454645456..924356faaecce6968b069f5ddacfecf6259930ca
100644
--- a/content/browser/resources/media/dump_creator.js
+++ b/content/browser/resources/media/dump_creator.js
@@ -89,9 +89,13 @@ var DumpCreator = (function() {
      * @private
      */
     onDownloadData_: function() {
-      var textBlob =
-          new Blob([JSON.stringify(peerConnectionDataStore, null, ' ')],
-                                   {type: 'octet/stream'});
+      var dump_object =
+      {
+        'getUserMedia': userMediaRequests,
+        'PeerConnections': peerConnectionDataStore,
+      };
+      var textBlob = new Blob([JSON.stringify(dump_object, null, ' ')],
+                              {type: 'octet/stream'});
       var URL = window.webkitURL.createObjectURL(textBlob);
       this.root_.getElementsByTagName('a')[0].href = URL;
       // The default action of the anchor will download the URL.

[commit-bot: I haz the power, 2014-01-10 19:48:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jiayl@chromium.org/129833002/100001

[commit-bot: I haz the power, 2014-01-10 23:23:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jiayl@chromium.org/129833002/100001

[commit-bot: I haz the power, 2014-01-11 00:12:32]

Change committed as 244281