Add link to Gold trybot runs in every uploaded Skia CL

PRESUBMIT.py

 # 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.
 
 
 """Top-level presubmit script for Skia.
 
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into gcl.
 """
@@ skipping 18 matching lines @@
     'reed@google.com',
     'bsalomon@chromium.org',
     'bsalomon@google.com',
     'djsollen@chromium.org',
     'djsollen@google.com',
 )
 
 AUTHORS_FILE_NAME = 'AUTHORS'
 
 DOCS_PREVIEW_URL = 'https://skia.org/?cl='
+GOLD_TRYBOT_URL = ('https://gold.skia.org/search2?unt=true'
+                   '&query=source_type%3Dgm&master=false&issue=')
 
 # Path to CQ bots feature is described in https://bug.skia.org/4364
 PATH_PREFIX_TO_EXTRA_TRYBOTS = {
     # pylint: disable=line-too-long
     'cmake/': 'client.skia.compile:Build-Mac10.9-Clang-x86_64-Release-CMake-Trybot,Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot',
     # pylint: disable=line-too-long
     'src/opts/': 'client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot',
 
     'include/private/SkAtomics.h': ('client.skia:'
       'Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-TSAN-Trybot,'
@@ skipping 291 matching lines @@
             "and add one of those reviewers on a TBR= line.  If you don't know "
             "if this CL affects Skia's public API, treat it like it does."
             % str(PUBLIC_API_OWNERS)))
   return results
 
 
 def PostUploadHook(cl, change, output_api):
   """git cl upload will call this hook after the issue is created/modified.
 
   This hook does the following:
+  * Adds a link to the CL's Gold trybot results.
   * Adds a link to preview docs changes if there are any docs changes in the CL.
   * Adds 'NOTRY=true' if the CL contains only docs changes.
   * Adds 'NOTREECHECKS=true' for non master branch changes since they do not
     need to be gated on the master branch's tree.
   * Adds 'NOTRY=true' for non master branch changes since trybots do not yet
     work on them.
   * Adds 'NOPRESUBMIT=true' for non master branch changes since those don't
     run the presubmit checks.
   * Adds extra trybots for the paths defined in PATH_TO_EXTRA_TRYBOTS.
   """
@@ skipping 10 matching lines @@
       all_docs_changes = False
     if atleast_one_docs_change and not all_docs_changes:
       break
 
   issue = cl.issue
   rietveld_obj = cl.RpcServer()
   if issue and rietveld_obj:
     original_description = rietveld_obj.get_description(issue)
     new_description = original_description
 
+    # Add GOLD_TRYBOT_URL if it does not exist yet.
+    if not re.search(r'^GOLD_TRYBOT_URL=', new_description, re.M | re.I):
+      new_description += '\nGOLD_TRYBOT_URL= %s%s' % (GOLD_TRYBOT_URL, issue)
+      results.append(
+          output_api.PresubmitNotifyResult(
+              'Added link to Gold trybot runs to the CL\'s description.\n'
+              'Note: Results may take sometime to be populated after trybots '
+              'complete.'))
+
     # If the change includes only doc changes then add NOTRY=true in the
     # CL's description if it does not exist yet.
     if all_docs_changes and not re.search(
         r'^NOTRY=true$', new_description, re.M | re.I):
       new_description += '\nNOTRY=true'
       results.append(
           output_api.PresubmitNotifyResult(
               'This change has only doc changes. Automatically added '
               '\'NOTRY=true\' to the CL\'s description'))
 
@@ skipping 114 matching lines @@
     state and an error if it is in 'Closed' state.
   """
   results = []
   results.extend(_CommonChecks(input_api, output_api))
   results.extend(
       _CheckTreeStatus(input_api, output_api, json_url=(
           SKIA_TREE_STATUS_URL + '/banner-status?format=json')))
   results.extend(_CheckLGTMsForPublicAPI(input_api, output_api))
   results.extend(_CheckOwnerIsInAuthorsFile(input_api, output_api))
   return results