[Telemetry] Always use ephemeral port chosen by Chrome.

tools/telemetry/telemetry/core/browser_options.py

 # Copyright 2012 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.
 
 import copy
 import logging
 import optparse
 import os
 import shlex
 import sys
@@ skipping 185 matching lines @@
     self.no_proxy_server = False
     self.browser_user_agent_type = None
 
     self.clear_sytem_cache_for_browser_and_profile_on_start = False
     self.startup_url = 'about:blank'
 
     # Background pages of built-in component extensions can interfere with
     # performance measurements.
     self.disable_component_extensions_with_background_pages = True
 
-    # Whether to use the new code path for choosing an ephemeral port for
-    # DevTools. The bots set this to true. When Chrome 37 reaches stable,
-    # remove this setting and the old code path. http://crbug.com/379980
-    self.use_devtools_active_port = False
-
   def __repr__(self):
     return str(sorted(self.__dict__.items()))
 
   @classmethod
   def AddCommandLineArgs(cls, parser):
 
     ############################################################################
     # Please do not add any more options here without first discussing with    #
     # a telemetry owner. This is not the right place for platform-specific     #
     # options.                                                                 #
@@ skipping 22 matching lines @@
     group.add_option('--netsim', default=None, type='choice',
         choices=net_configs.NET_CONFIG_NAMES,
         help=('Run benchmark under simulated network conditions. '
               'Will prompt for sudo. Supported values: ' +
               ', '.join(net_configs.NET_CONFIG_NAMES)))
     group.add_option('--show-stdout',
         action='store_true',
         help='When possible, will display the stdout of the process')
     # This hidden option is to be removed, and the older code path deleted,
     # once Chrome 37 reaches Stable. http://crbug.com/379980
     group.add_option('--use-devtools-active-port',

[Ken Russell (switch to Gerrit), 2014/10/14 05:57:14]

We should remove this too after landing this CL and after updating all the
recipes to no longer pass this argument.

         action='store_true',
         help=optparse.SUPPRESS_HELP)
     parser.add_option_group(group)
 
     group = optparse.OptionGroup(parser, 'Compatibility options')
     group.add_option('--gtest_output',
         help='Ignored argument for compatibility with runtest.py harness')
     parser.add_option_group(group)
 
     group = optparse.OptionGroup(parser, 'Synthetic gesture options')
@@ skipping 12 matching lines @@
   def UpdateFromParseResults(self, finder_options):
     """Copies our options from finder_options"""
     browser_options_list = [
         'extra_browser_args_as_string',
         'extra_wpr_args_as_string',
         'netsim',
         'profile_dir',
         'profile_type',
         'show_stdout',
         'synthetic_gesture_source_type',
-        'use_devtools_active_port',
         ]
     for o in browser_options_list:
       a = getattr(finder_options, o, None)
       if a is not None:
         setattr(self, o, a)
         delattr(finder_options, o)
 
     self.browser_type = finder_options.browser_type
 
     if hasattr(self, 'extra_browser_args_as_string'): # pylint: disable=E1101
@@ skipping 33 matching lines @@
 
   @property
   def extra_browser_args(self):
     return self._extra_browser_args
 
   def AppendExtraBrowserArgs(self, args):
     if isinstance(args, list):
       self._extra_browser_args.update(args)
     else:
       self._extra_browser_args.add(args)