Adds the UserMedia requests to the UI of chrome://webrtc-internals.

Adds the UserMedia requests to the UI of chrome://webrtc-internals.
Also fix an issue with missing constriants logged at webrtc-internals.

See a snapshot of the UI at https://docs.google.com/a/google.com/presentation/d/1dL2flqsQj8h_dTO9t8BWOCjiwbLiZCN2Nt93FHqpM9o/edit?usp=sharing

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

[jiayl, 2014-07-18 21:48:42]

vrk,

Could you take a look?

[vrk (LEFT CHROMIUM), 2014-07-18 22:01:06]

lgtm

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

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

https://codereview.chromium.org/405683003/diff/1/content/browser/resources/me...
content/browser/resources/media/webrtc_internals.js:318: if
($(USER_MEDIA_TAB_ID).childNodes.length == 0) {
nit: no {}

[jiayl, 2014-07-18 22:06:24]

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

https://codereview.chromium.org/405683003/diff/1/content/browser/resources/me...
content/browser/resources/media/webrtc_internals.js:314: if (requests[i].rid ==
data.rid) {
On 2014/07/18 22:01:06, Victoria Kirst wrote:
> nit: no {}

Done.

https://codereview.chromium.org/405683003/diff/1/content/browser/resources/me...
content/browser/resources/media/webrtc_internals.js:318: if
($(USER_MEDIA_TAB_ID).childNodes.length == 0) {
On 2014/07/18 22:01:06, Victoria Kirst wrote:
> nit: no {}

Done.

[jiayl, 2014-07-18 22:06:27]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2014-07-18 22:08:51]

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

[commit-bot: I haz the power, 2014-07-19 01:51:48]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  linux_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu_trigger...)
  mac_gpu_retina_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_retina_tr...)
  mac_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_triggered...)
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-07-19 02:12:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-19 02:12:38]

Try jobs failed on following builders:
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)
  mac_gpu_retina_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_retina_tr...)

[jiayl, 2014-07-21 15:53:02]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2014-07-21 15:53:54]

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

[commit-bot: I haz the power, 2014-07-21 17:27:30]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_rel_swarming on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel_sw...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  linux_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu_trigger...)
  mac_gpu_retina_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_retina_tr...)
  mac_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_triggered...)
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-07-21 17:47:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-21 17:47:01]

Try jobs failed on following builders:
  linux_chromium_rel_swarming on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel_sw...)

[jiayl, 2014-07-21 19:44:35]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2014-07-21 19:48:32]

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

[jiayl, 2014-07-21 21:11:20]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2014-07-21 21:12:53]

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

[commit-bot: I haz the power, 2014-07-22 00:07:37]

Change committed as 284545