Re-enable the multiple_peerconnections story and disable webrtc traces and metrics.

tools/perf/page_sets/webrtc_cases.py

 # Copyright 2014 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.
 
 from telemetry import story
 from telemetry.page import page as page_module
 
 
+CODECS = ('OPUS', 'G772', 'PCMU', 'ISAC/1600')
+
+
 class WebrtcPage(page_module.Page):
 
   def __init__(self, url, page_set, name, tags):
     assert url.startswith('file://webrtc_cases/')
     super(WebrtcPage, self).__init__(
         url=url, page_set=page_set, name=name, tags=tags)
 
 
 class GetUserMedia(WebrtcPage):
   """Why: Acquires a high definition (720p) local stream."""
@@ skipping 84 matching lines @@
 
   def __init__(self, page_set, tags):
     super(MultiplePeerConnections, self).__init__(
         url='file://webrtc_cases/multiple-peerconnections.html',
         name='multiple_peerconnections',
         page_set=page_set, tags=tags)
 
   def RunPageInteractions(self, action_runner):
     with action_runner.CreateInteraction('Action_Create_PeerConnection',
                                          repeatable=False):
-      # Set the number of peer connections to create to 15.
+      # Set the number of peer connections to create to 10.
       action_runner.ExecuteJavaScript(
-          'document.getElementById("num-peerconnections").value=15')
+          'document.getElementById("num-peerconnections").value=10')
       action_runner.ExecuteJavaScript(
           'document.getElementById("cpuoveruse-detection").checked=false')
       action_runner.ClickElement('button[id="start-test"]')
-      action_runner.Wait(45)
+      action_runner.Wait(20)
 
 
 class WebrtcPageSet(story.StorySet):
   def __init__(self):
     super(WebrtcPageSet, self).__init__(
         cloud_storage_bucket=story.PUBLIC_BUCKET)
 
+    self.AddStory(MultiplePeerConnections(self, tags=['stress']))
+    self.AddStory(DataChannel(self, tags=['datachannel']))
     self.AddStory(GetUserMedia(self, tags=['getusermedia']))
-    self.AddStory(MultiplePeerConnections(self, tags=['stress']))
     self.AddStory(VideoCall(self, tags=['peerconnection', 'smoothness']))
-    self.AddStory(DataChannel(self, tags=['datachannel']))
     self.AddStory(CanvasCapturePeerConnection(self, tags=['smoothness']))
-    # TODO(qyearsley, mcasas): Add webrtc.audio when http://crbug.com/468732
-    # is fixed, or revert https://codereview.chromium.org/1544573002/ when
-    # http://crbug.com/568333 is fixed.
-    # self.AddStory(AudioCall(self, 'OPUS'))
-    # self.AddStory(AudioCall(self, 'G772'))
-    # self.AddStory(AudioCall(self, 'PCMU'))
-    # self.AddStory(AudioCall(self, 'ISAC/1600'))
+    for codec in CODECS:

[nednguyen, 2017/05/26 20:17:07]

I also don't think code saving here is a good idea.

+      self.AddStory(AudioCall(self, codec, tags=['audio']))
 
 
 class WebrtcExpectations(story.expectations.StoryExpectations):
   def SetExpectations(self):
-    self.DisableStory('multiple_peerconnections',
-                      [story.expectations.ALL],
-                      'crbug.com/725502')
+    # TODO(qyearsley, mcasas): Add webrtc.audio when http://crbug.com/468732
+    # is fixed, or revert https://codereview.chromium.org/1544573002/ when
+    # http://crbug.com/568333 is fixed.
+    for codec in CODECS:

[nednguyen, 2017/05/26 20:16:09]

No, please don't do this as it makes sheriffs super confused about which stories
are disabled & which are not. You should just manually list out all the story
that are disabled.

+      self.DisableStory('audio_call_%s_10s' % codec.lower(),
+                        [story.expectations.ALL],
+                        'crbug.com/468732')
     self.DisableStory('30s_datachannel_transfer',
                       [story.expectations.ALL_DESKTOP],
                       'crbug.com/726811')