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

content/browser/resources/media/dump_creator.js

 // Copyright (c) 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.
 
 
 /**
  * Provides the UI for dump creation.
  */
 var DumpCreator = (function() {
   /**
@@ skipping 37 matching lines @@
         '.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><label><input type=checkbox>' +
         'Enable diagnostic packet and event recording</label></p>' +
         '<p class=audio-recordings-info>A diagnostic packet and event' +

[Henrik Grunell, 2016/05/13 09:05:31]

Nit: The class name "audio-recordings-info" isn't accurate longer as it has
expanded to event log. Would you mind changing it to something more suitable?

[Ivo-OOO until feb 6, 2016/05/18 16:26:55]

Done.

         ' 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 selected. The entered filename is used as a' +

[Henrik Grunell, 2016/05/13 09:05:31]

Nit: "entered" -> "selected".

[Ivo-OOO until feb 6, 2016/05/18 16:26:54]

Done.

+        ' base, to which the following suffixes will be appended.</p>' +
+        ' <p><div>&lt;base filename&gt;.&lt;render process ID&gt;' +

[Henrik Grunell, 2016/05/13 09:05:31]

Nit: No need for <div></div> since there's only one in the <p></p>.

[Ivo-OOO until feb 6, 2016/05/18 16:26:55]

Done.

+        '.&lt;recording ID&gt;</div></p>' +
+        '<p class=audio-recordings-info>Event logs from different tabs will' +

[Henrik Grunell, 2016/05/13 09:05:31]

This is not always true; different tabs can share render process in some cases.
You could just skip that part or rephrase.

[Ivo-OOO until feb 6, 2016/05/18 16:26:55]

I did not know that. I think removing this part would be better otherwise it
gets a bit confusing.

+        ' have a different render process ID, and recordings from multiple' +
+        ' peerconnections within the same tab will have a different recording' +

[Henrik Grunell, 2016/05/13 09:05:31]

Nit: PeerConnections

[Henrik Grunell, 2016/05/13 09:05:31]

"...will have different recording IDs."

[Ivo-OOO until feb 6, 2016/05/18 16:26:55]

This text is removed now.

+        ' ID. If a file with the same name already exists, it will be' +
+        ' overwritten. </p>';
     content.getElementsByTagName('a')[0].addEventListener(
         'click', this.onDownloadData_.bind(this));
     content.getElementsByTagName('input')[0].addEventListener(
         'click', this.onAudioDebugRecordingsChanged_.bind(this));
     content.getElementsByTagName('input')[1].addEventListener(
         'click', this.onEventLogRecordingsChanged_.bind(this));
   }
 
   DumpCreator.prototype = {
     // Mark the diagnostic audio recording checkbox checked.
@@ skipping 60 matching lines @@
       var enabled = this.root_.getElementsByTagName('input')[1].checked;
       if (enabled) {
         chrome.send('enableEventLogRecordings');
       } else {
         chrome.send('disableEventLogRecordings');
       }
     },
   };
   return DumpCreator;
 })();