Add logcats link in builbot page for android swarming tasks

scripts/slave/recipe_modules/chromium_tests/steps.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.
 
 import datetime
 import re
 import string
-
+import urllib
 
 class Test(object):
   """
   Base class for tests that can be retried after deapplying a previously
   applied patch.
   """
 
   def __init__(self):
     super(Test, self).__init__()
     self._test_runs = {}
@@ skipping 886 matching lines @@
 
     return True, gtest_results.failures
 
   def post_run(self, api, suffix, test_filter=None):
     """Waits for launched test to finish and collects the results."""
     try:
       super(SwarmingGTestTest, self).post_run(
           api, suffix,test_filter=test_filter)
     finally:
       step_result = api.step.active_result
+
+      # If Android swarming tasks, create link for unified logcats
+      # https://crbug.com/448050
+      if api.chromium.c.TARGET_PLATFORM == 'android':
+        for task in self._tasks.values():
+          if hasattr(task, 'trigger_output') and 'tasks' in task.trigger_output:

[martiniss, 2016/08/04 00:44:09]

If we don't have this, what do we do? Is it an error? I would think so, right?
We should handle that, and output something like "logcat missing, file a bug?"
or something. Does that make sense?

[nicholaslin, 2016/08/04 19:28:51]

Added in an additional check to make sure the logdog butler cipd package is
installed. If the butler package is present and there is an output I think its
safe to assume the task is written to logdog?

+            for test in task.trigger_output['tasks'].values():
+              task_id = test.get('task_id')
+              shard_index = test.get('shard_index')
+              if shard_index is not None and task_id is not None:
+                prefix = 'https://luci-logdog.appspot.com/v/?s='
+                suffix = '%s/%s/%s/%s/+/%s' % ('android', 'swarming', 'logcats',
+                                               task_id, 'unified_logcats')
+                step_result.presentation.links[('shard_index: %d logcats' %
+                                                shard_index)] = (
+                    prefix + urllib.quote_plus(suffix))
+
       # Only upload test results if we have gtest results.
       if (self._upload_test_results and
           hasattr(step_result, 'test_utils') and
           hasattr(step_result.test_utils, 'gtest_results')):
         gtest_results = getattr(step_result.test_utils, 'gtest_results', None)
         if gtest_results and gtest_results.raw:
           parsed_gtest_data = gtest_results.raw
           chrome_revision_cp = api.bot_update.last_returned_properties.get(
               'got_revision_cp', 'x@{#0}')
           chrome_revision = str(api.commit_position.parse_revision(
@@ skipping 819 matching lines @@
           args=args)
       api.gsutil.upload(
           temp_output_dir.join(
               '%s-android-chrome.json' % timestamp_string),
           'chromium-annotated-tests', 'android')
 
 GOMA_TESTS = [
   GTestTest('base_unittests'),
   GTestTest('content_unittests'),
 ]