Update documentation for new perf bisect command.

tools/bisect-perf-regression.py

 #!/usr/bin/env python
 # 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.
 
 """Performance Test Bisect Tool
 
 This script bisects a series of changelists using binary search. It starts at
 a bad revision where a performance metric has regressed, and asks for a last
 known-good revision. It will then binary search across this revision range by
@@ skipping 217 matching lines @@
 committed locally to run-perf-test.cfg.
    Note: *DO NOT* commit run-perf-test.cfg changes to the project repository.
    $ git cl upload --bypass-hooks
 4. Send your try job to the try server. \
 [Please make sure to use appropriate bot to reproduce]
    $ git cl try -m tryserver.chromium.perf -b <bot>
 
 For more details please visit
 https://sites.google.com/a/chromium.org/dev/developers/performance-try-bots"""
 
+REPRO_STEPS_TRYJOB_TELEMETRY = """
+To reproduce on a performance try bot:
+%(command)s
+(Where <bot-name> comes from tools/perf/run_benchmark --browser=list)

[tonyg, 2014/08/27 16:38:59]

I think an important thing to do here is to give them the bot name rather than
asking them to find it. We already know exactly which bot this bisect ran on and
the problem we've had in the past is that authors didn't understand that the
regression was platform specific, thus leading to confusion like in
https://docs.google.com/a/google.com/document/d/1GpQI97nMsye57o6JNLyFSpvzY0fL...

[sullivan, 2014/08/27 22:02:49]

I'm really sorry this got committed without addressing your comment. Feel free
to revert while I am on vacation.

I agree it would be much better to print the name of the bisect bot. But I
wasn't able to find the name.

There is the function GetBuilderNameAndBuildTime(), but it looks like it returns
the builder, not the tryserver.

There is builder_host, but I think that is an http hostname and not the
buildername, right?

Prasad, Tony, any ideas how I can get the tryserver name, e.g.
mac_10_9_perf_bisect?

+
+For more details please visit
+https://sites.google.com/a/chromium.org/dev/developers/performance-try-bots
+"""
+
 RESULTS_THANKYOU = """
 ===== THANK YOU FOR CHOOSING BISECT AIRLINES =====
 Visit http://www.chromium.org/developers/core-principles for Chrome's policy
 on perf regressions.
 Contact chrome-perf-dashboard-team with any questions or suggestions about
 bisecting.
 .                   .------.
 .     .---.         \       \==)
 .     |PERF\         \       \\
 .     |     ---------'-------'-----------.
@@ skipping 2470 matching lines @@
   def _PrintReproSteps(self):
     """Prints out a section of the results explaining how to run the test.
 
     This message includes the command used to run the test.
     """
     command = '$ ' + self.opts.command
     if bisect_utils.IsTelemetryCommand(self.opts.command):
       command += ('\nAlso consider passing --profiler=list to see available '
                   'profilers.')
     print REPRO_STEPS_LOCAL % {'command': command}
-    print REPRO_STEPS_TRYJOB % {'command': command}
+    if bisect_utils.IsTelemetryCommand(self.opts.command):
+      telemetry_command = re.sub(r'--browser=[^\s]+',
+                                 '--browser=<bot-name>',
+                                 command)
+      print REPRO_STEPS_TRYJOB_TELEMETRY % {'command': telemetry_command}
+    else:
+      print REPRO_STEPS_TRYJOB % {'command': command}
 
   def _PrintOtherRegressions(self, other_regressions, revision_data):
     """Prints a section of the results about other potential regressions."""
     print
     print 'Other regressions may have occurred:'
     print '  %8s  %70s  %10s' % ('Depot'.center(8, ' '),
         'Range'.center(70, ' '), 'Confidence'.center(10, ' '))
     for regression in other_regressions:
       current_id, previous_id, confidence = regression
       current_data = revision_data[current_id]
@@ skipping 623 matching lines @@
       # bugs. If you change this, please update the perf dashboard as well.
       bisect_utils.OutputAnnotationStepStart('Results')
     print 'Error: %s' % e.message
     if opts.output_buildbot_annotations:
       bisect_utils.OutputAnnotationStepClosed()
   return 1
 
 
 if __name__ == '__main__':
   sys.exit(main())