Store tombstones url, not the actual tombstones, inside test result.

build/android/pylib/local/device/local_device_gtest_run.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 collections
 import itertools
 import logging
 import os
 import posixpath
+import time
 
 from devil.android import device_errors
 from devil.android import device_temp_file
 from devil.android import ports
 from devil.utils import reraiser_thread
 from pylib import constants
 from pylib.base import base_test_result
 from pylib.gtest import gtest_test_instance
 from pylib.local import local_test_server_spawner
 from pylib.local.device import local_device_environment
@@ skipping 387 matching lines @@
     if self._test_instance.enable_xml_result_parsing:
       results = gtest_test_instance.ParseGTestXML(gtest_xml)
     else:
       results = gtest_test_instance.ParseGTestOutput(output)
 
     # Check whether there are any crashed testcases.
     self._crashes.update(r.GetName() for r in results
                          if r.GetType() == base_test_result.ResultType.CRASH)
 
     if self._test_instance.store_tombstones:
       resolved_tombstones = None

[jbudorick, 2016/12/15 22:31:35]

Should we switch this & the if check below to tombstones_url?

[BigBossZhiling, 2016/12/20 19:34:08]

Done.

       for result in results:
         if result.GetType() == base_test_result.ResultType.CRASH:
           if not resolved_tombstones:
-            resolved_tombstones = '\n'.join(tombstones.ResolveTombstones(
+            resolved_tombstones = tombstones.ResolveTombstones(
                 device,
                 resolve_all_tombstones=True,
                 include_stack_symbols=False,
-                wipe_tombstones=True))
-          result.SetTombstones(resolved_tombstones)
+                wipe_tombstones=True)
+            stream_name = 'tombstones_%s_%s_%s' % (

[jbudorick, 2016/12/15 22:31:35]

That said, this stream name may be confusing if we first have a crash for test A
that then gets associated w/ a test B.

[BigBossZhiling, 2016/12/20 19:34:08]

Done.

+                result.GetName(),
+                time.strftime('%Y%m%dT%H%M%S', time.localtime()),
+                device.serial)
+            tombstones_url = tombstones.LogdogTombstones(resolved_tombstones,
+                                                         stream_name)
+          result.SetTombstonesUrl(tombstones_url)
 
     not_run_tests = set(test).difference(set(r.GetName() for r in results))
     return results, list(not_run_tests)
 
   #override
   def TearDown(self):
     @local_device_environment.handle_shard_failures
     def individual_device_tear_down(dev):
       for s in self._servers.get(str(dev), []):
         s.TearDown()
 
       tool = self.GetTool(dev)
       tool.CleanUpEnvironment()
 
     self._env.parallel_devices.pMap(individual_device_tear_down)