Revert of Telemetry: Last ditch effort to dump symbols on tab crashes if none...

Revert of Telemetry: Last ditch effort to dump symbols on tab crashes if none are found (https://codereview.chromium.org/318123002/)

Reason for revert:
page_runner_unittest.PageRunnerTests.testHandlingOfCrashedTab is failing on the main waterfall after this change:

http://build.chromium.org/p/chromium.mac/builders/Mac10.6%20Tests%20%283%29/builds/48907

http://build.chromium.org/p/chromium.mac/builders/Mac10.7%20Tests%20%283%29/builds/27652

http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests/builds/11451

Original issue's description:
> Telemetry: Last ditch effort to dump symbols on tab crashes if none are found
> 
> Put this together to find a crash which only happened on telemetry unittest runs on Mac trybots; figured it might be useful for later too.
> 
> R=dtu,tonyg
> BUG=
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275675

TBR=dtu@chromium.org,tonyg@chromium.org,oysteine@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=

Review URL: https://codereview.chromium.org/320693002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275678 0039d316-1c4b-4281-b951-d872f2087c98

tools/telemetry/telemetry/core/backends/chrome/desktop_browser_backend.py

@@ -273,47 +273,31 @@ class DesktopBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
       logging.warning('minidump_stackwalk binary not found.')
       return None
 
+    symbols = glob.glob(os.path.join(self._browser_directory, '*.breakpad*'))
+    if not symbols:
+      logging.warning('No breakpad symbols found.')
+      return None
+
     with open(minidump, 'rb') as infile:
       minidump += '.stripped'
       with open(minidump, 'wb') as outfile:
         outfile.write(''.join(infile.read().partition('MDMP')[1:]))
 
     symbols_path = os.path.join(self._tmp_minidump_dir, 'symbols')
-
-    symbols = glob.glob(os.path.join(self._browser_directory, '*.breakpad*'))
-    if symbols:
-      for symbol in sorted(symbols, key=os.path.getmtime, reverse=True):
-        if not os.path.isfile(symbol):
+    for symbol in sorted(symbols, key=os.path.getmtime, reverse=True):
+      if not os.path.isfile(symbol):
+        continue
+      with open(symbol, 'r') as f:
+        fields = f.readline().split()
+        if not fields:
           continue
-        with open(symbol, 'r') as f:
-          fields = f.readline().split()
-          if not fields:
-            continue
-          sha = fields[3]
-          binary = ' '.join(fields[4:])
-        symbol_path = os.path.join(symbols_path, binary, sha)
-        if os.path.exists(symbol_path):
-          continue
-        os.makedirs(symbol_path)
-        shutil.copyfile(symbol, os.path.join(symbol_path, binary + '.sym'))
-    else:
-      logging.info('Dumping breakpad symbols')
-      generate_breakpad_symbols_path = os.path.join(
-          util.GetChromiumSrcDir(), "components", "breakpad",
-          "tools", "generate_breakpad_symbols.py")
-      cmd = [
-          sys.executable,
-          generate_breakpad_symbols_path,
-          '--binary=%s' % self._executable,
-          '--symbols-dir=%s' % symbols_path,
-          '--build-dir=%s' % self._browser_directory,
-          ]
-
-      try:
-        subprocess.check_output(cmd, stderr=open(os.devnull, 'w'))
-      except subprocess.CalledProcessError:
-        logging.warning('Failed to execute "%s"' % ' '.join(cmd))
-        return None
+        sha = fields[3]
+        binary = ' '.join(fields[4:])
+      symbol_path = os.path.join(symbols_path, binary, sha)
+      if os.path.exists(symbol_path):
+        continue
+      os.makedirs(symbol_path)
+      shutil.copyfile(symbol, os.path.join(symbol_path, binary + '.sym'))
 
     return subprocess.check_output([stackwalk, minidump, symbols_path],
                                    stderr=open(os.devnull, 'w'))