Move the call to enable the WebRTC event log from PeerConnectionFactory to PeerConnection.

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 c8f53dfc165c69a2765153145451c6c7c4bad818..a3e2937e231c108621c846dd214d85c8029060c7 100644
--- a/content/browser/resources/media/dump_creator.js
+++ b/content/browser/resources/media/dump_creator.js
@@ -33,12 +33,12 @@ var DumpCreator = (function() {
         '</button></a></div>' +
         '<p><label><input type=checkbox>' +
         'Enable diagnostic audio recordings</label></p>' +
-        '<p class=audio-recordings-info>A diagnostic audio recording is used' +
-        ' for analyzing audio problems. It consists of two files and contains' +
-        ' the audio played out from the speaker and recorded from the' +
-        ' microphone and is saved to the local disk. Checking this box will' +
-        ' enable the recording for ongoing WebRTC calls and for future WebRTC' +
-        ' calls. When the box is unchecked or this page is closed, all' +
+        '<p class=audio-diagnostic-dumps-info>A diagnostic audio recording is' +
+        ' used for analyzing audio problems. It consists of two files and' +
+        ' contains the audio played out from the speaker and recorded from' +
+        ' the microphone and is saved to the local disk. Checking this box' +
+        ' will enable the recording for ongoing WebRTC calls and for future' +
+        ' WebRTC calls. When the box is unchecked or this page is closed, all' +
         ' ongoing recordings will be stopped and this recording' +
         ' functionality will be disabled for future WebRTC calls. Recordings' +
         ' in multiple tabs are supported as well as multiple recordings in' +
@@ -48,26 +48,34 @@ var DumpCreator = (function() {
         '.aec_dump.&lt;recording ID&gt;</div>' +
         '<div>&lt;base filename&gt;.&lt;render process ID&gt;' +
         '.source_input.&lt;stream ID&gt;.wav</div></p>' +
-        '<p class=audio-recordings-info>If recordings are disabled and then' +
-        ' enabled using the same base filename, the microphone recording file' +
-        ' will be overwritten, and the AEC dump file will be appended to and' +
-        ' may become invalid. It is recommended to choose a new base filename' +
-        ' each time or move the produced files before enabling again.</p>' +
+        '<p class=audio-diagnostic-dumps-info>If recordings are disabled and' +
+        ' then enabled using the same base filename, the microphone recording' +
+        ' file will be overwritten, and the AEC dump file will be appended to' +
+        ' and may become invalid. It is recommended to choose a new base' +
+        ' filename each time or move the produced files before enabling' +
+        ' again.</p>' +
         '<p><label><input type=checkbox>' +
         'Enable diagnostic packet and event recording</label></p>' +
-        '<p class=audio-recordings-info>A diagnostic packet and event' +
+        '<p class=audio-diagnostic-dumps-info>A diagnostic packet and event' +
         ' recording can be used for analyzing various issues related to' +
         ' thread starvation, jitter buffers or bandwidth estimation. Two' +
         ' types of data are logged. First, incoming and outgoing RTP headers' +
         ' and RTCP packets are logged. These do not include any audio or' +
         ' video information, nor any other types of personally identifiable' +
         ' information (so no IP addresses or URLs). Checking this box will' +
-        ' enable the recording for currently ongoing WebRTC calls. When' +
-        ' the box is unchecked or this page is closed, all active recordings' +
-        ' will be stopped. Recording in multiple tabs or multiple recordings' +
-        ' in the same tab is currently not supported. When enabling, a' +
-        ' filename for the recording can be selected. If an existing file is' +
-        ' selected, it will be overwritten. </p>';
+        ' enable the recording for ongoing WebRTC calls and for future' +
+        ' WebRTC calls. When the box is unchecked or this page is closed,' +
+        ' all ongoing recordings will be stopped and this recording' +
+        ' functionality will be disabled for future WebRTC calls. Recording' +
+        ' in multiple tabs or multiple recordings in the same tab will cause' +
+        ' multiple log files to be created. When enabling, a filename for the' +
+        ' recording can be entered. The entered filename is used as a' +
+        ' base, to which the following suffixes will be appended.</p>' +
+        ' <p>&lt;base filename&gt;.&lt;render process ID&gt;' +
+        '.&lt;recording ID&gt;</p>' +
+        '<p class=audio-diagnostic-dumps-info>If a file with the same name' +
+        ' already exists, it will be' +
+        ' overwritten. </p>';

[Henrik Grunell, 2016/06/01 08:31:11]

Overwritten or appended?

[Henrik Grunell, 2016/06/01 08:31:11]

Nit: fits on above line.

[Ivo-OOO until feb 6, 2016/06/02 14:43:39]

Since I changed it to actually overwrite, the text is now correct :-)

[Henrik Grunell, 2016/06/03 07:40:52]

Acknowledged.

     content.getElementsByTagName('a')[0].addEventListener(
         'click', this.onDownloadData_.bind(this));
     content.getElementsByTagName('input')[0].addEventListener(