chore: remove no longer needed debugging patch (#23030)
This commit is contained in:
parent
ca947307db
commit
23ae0e6bd8
2 changed files with 0 additions and 85 deletions
|
@ -48,7 +48,6 @@ dump_syms.patch
|
||||||
command-ismediakey.patch
|
command-ismediakey.patch
|
||||||
tts.patch
|
tts.patch
|
||||||
printing.patch
|
printing.patch
|
||||||
verbose_generate_breakpad_symbols.patch
|
|
||||||
support_mixed_sandbox_with_zygote.patch
|
support_mixed_sandbox_with_zygote.patch
|
||||||
autofill_size_calculation.patch
|
autofill_size_calculation.patch
|
||||||
fix_disable_usage_of_abort_report_np_in_mas_builds.patch
|
fix_disable_usage_of_abort_report_np_in_mas_builds.patch
|
||||||
|
|
|
@ -1,84 +0,0 @@
|
||||||
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
||||||
From: John Kleinschmidt <jkleinsc@github.com>
|
|
||||||
Date: Thu, 18 Oct 2018 17:08:28 -0700
|
|
||||||
Subject: verbose_generate_breakpad_symbols.patch
|
|
||||||
|
|
||||||
Temporarily add additional debugging statements to
|
|
||||||
generate_breakpad_symbols.py to determine why it is hanging.
|
|
||||||
|
|
||||||
diff --git a/components/crash/content/tools/generate_breakpad_symbols.py b/components/crash/content/tools/generate_breakpad_symbols.py
|
|
||||||
index a367fa2bf31591c98dd4245d42a71cd9da82f2d9..7ecc72117881fdfa64170d62f94627a7f6dda7ad 100755
|
|
||||||
--- a/components/crash/content/tools/generate_breakpad_symbols.py
|
|
||||||
+++ b/components/crash/content/tools/generate_breakpad_symbols.py
|
|
||||||
@@ -60,7 +60,10 @@ def Resolve(path, exe_path, loader_path, rpaths):
|
|
||||||
return path
|
|
||||||
|
|
||||||
|
|
||||||
-def GetSharedLibraryDependenciesLinux(binary):
|
|
||||||
+def GetSharedLibraryDependenciesLinux(binary, options):
|
|
||||||
+ if options.verbose:
|
|
||||||
+ print "GetSharedLibraryDependencies for %s" % binary
|
|
||||||
+
|
|
||||||
"""Return absolute paths to all shared library dependencies of the binary.
|
|
||||||
|
|
||||||
This implementation assumes that we're running on a Linux system."""
|
|
||||||
@@ -71,6 +74,9 @@ def GetSharedLibraryDependenciesLinux(binary):
|
|
||||||
m = lib_re.match(line)
|
|
||||||
if m:
|
|
||||||
result.append(os.path.abspath(m.group(1)))
|
|
||||||
+ if options.verbose:
|
|
||||||
+ print "Done GetSharedLibraryDependencies for %s" % binary
|
|
||||||
+ print result
|
|
||||||
return result
|
|
||||||
|
|
||||||
|
|
||||||
@@ -215,7 +221,7 @@ def GetSharedLibraryDependencies(options, binary, exe_path):
|
|
||||||
"""Return absolute paths to all shared library dependencies of the binary."""
|
|
||||||
deps = []
|
|
||||||
if options.platform == 'linux2':
|
|
||||||
- deps = GetSharedLibraryDependenciesLinux(binary)
|
|
||||||
+ deps = GetSharedLibraryDependenciesLinux(binary, options)
|
|
||||||
elif options.platform == 'android':
|
|
||||||
deps = GetSharedLibraryDependenciesAndroid(binary)
|
|
||||||
elif options.platform == 'darwin':
|
|
||||||
@@ -292,7 +298,8 @@ def CreateSymbolDir(options, output_dir, relative_hash_dir):
|
|
||||||
|
|
||||||
def GenerateSymbols(options, binaries):
|
|
||||||
"""Dumps the symbols of binary and places them in the given directory."""
|
|
||||||
-
|
|
||||||
+ if options.verbose:
|
|
||||||
+ print "Generating symbols for %s " % (' '.join(binaries))
|
|
||||||
queue = Queue.Queue()
|
|
||||||
print_lock = threading.Lock()
|
|
||||||
|
|
||||||
@@ -312,8 +319,15 @@ def GenerateSymbols(options, binaries):
|
|
||||||
reason = "Could not locate dump_syms executable."
|
|
||||||
break
|
|
||||||
|
|
||||||
+ if options.verbose:
|
|
||||||
+ with print_lock:
|
|
||||||
+ print "GetBinaryInfoFromHeaderInfo for %s" % (binary)
|
|
||||||
binary_info = GetBinaryInfoFromHeaderInfo(
|
|
||||||
subprocess.check_output([dump_syms, '-i', binary]).splitlines()[0])
|
|
||||||
+ if options.verbose:
|
|
||||||
+ with print_lock:
|
|
||||||
+ print "Done GetBinaryInfoFromHeaderInfo for %s: %s (%s)" % (binary, binary_info.name, binary_info.hash)
|
|
||||||
+
|
|
||||||
if not binary_info:
|
|
||||||
should_dump_syms = False
|
|
||||||
reason = "Could not obtain binary information."
|
|
||||||
@@ -331,8 +345,14 @@ def GenerateSymbols(options, binaries):
|
|
||||||
# See if there is a symbol file already found next to the binary
|
|
||||||
potential_symbol_files = glob.glob('%s.breakpad*' % binary)
|
|
||||||
for potential_symbol_file in potential_symbol_files:
|
|
||||||
+ if options.verbose:
|
|
||||||
+ with print_lock:
|
|
||||||
+ print "Looking at %s as a potential symbol file." % (potential_symbol_file)
|
|
||||||
with open(potential_symbol_file, 'rt') as f:
|
|
||||||
symbol_info = GetBinaryInfoFromHeaderInfo(f.readline())
|
|
||||||
+ if options.verbose:
|
|
||||||
+ with print_lock:
|
|
||||||
+ print "Got symbol_info for %s." % (potential_symbol_file)
|
|
||||||
if symbol_info == binary_info:
|
|
||||||
CreateSymbolDir(options, output_dir, binary_info.hash)
|
|
||||||
shutil.copyfile(potential_symbol_file, output_path)
|
|
Loading…
Add table
Add a link
Reference in a new issue