Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(683)

Unified Diff: appengine/findit/crash/callstack.py

Issue 1914113002: [Findit] Enable project classifier and component classifier (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Rebase Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | appengine/findit/crash/classifier.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/crash/callstack.py
diff --git a/appengine/findit/crash/callstack.py b/appengine/findit/crash/callstack.py
index 5da479d354ab485da7c3359ae221b1001418db26..715f7ac56262bf8a64011f63b7fdebb0a2f197b2 100644
--- a/appengine/findit/crash/callstack.py
+++ b/appengine/findit/crash/callstack.py
@@ -29,18 +29,21 @@ class StackFrame(object):
this - '#0 ...', else use the index in the callstack list.
dep_path (str): Path of the dep this frame represents, for example,
'src/', 'src/v8', 'src/skia'...etc.
- component (str): Component of this frame, for example, 'Blink>API'.
function (str): Function that caused the crash.
- file_path (str): Path of the crashed file.
+ file_path (str): Normalized path of the crashed file, with parts dep_path
+ and parts before it stripped, for example, api.cc.
+ raw_file_path (str): Normalized original path of the crashed file,
+ for example, /b/build/slave/mac64/build/src/v8/src/heap/
+ incremental-marking-job.cc.
crashed_line_numbers (list): Line numbers of the file that caused the crash.
"""
- def __init__(self, index, dep_path, component,
- function, file_path, crashed_line_numbers):
+ def __init__(self, index, dep_path, function,
+ file_path, raw_file_path, crashed_line_numbers):
self.index = index
self.dep_path = dep_path
- self.component = component
self.function = function
self.file_path = file_path
+ self.raw_file_path = raw_file_path
self.crashed_line_numbers = crashed_line_numbers
def ToString(self):
@@ -98,7 +101,7 @@ class CallStack(list):
return
function = match.group(1)
- file_path = parse_util.GetFullPathForJavaFrame(function)
+ raw_file_path = parse_util.GetFullPathForJavaFrame(function)
crashed_line_numbers = [int(match.group(3))]
elif self.format_type == CallStackFormatType.SYZYASAN:
@@ -107,7 +110,7 @@ class CallStack(list):
return
function = match.group(2).strip()
- file_path = match.group(5)
+ raw_file_path = match.group(5)
crashed_line_numbers = [int(match.group(6))]
else:
@@ -120,16 +123,13 @@ class CallStack(list):
return
function = ' '.join(line_parts[3:-1])
- file_path = match.group(1)
+ raw_file_path = match.group(1)
crashed_line_numbers = parse_util.GetCrashedLineRange(
match.group(2) + (match.group(3) if match.group(3) else ''))
# Normalize the file path so that it can be compared to repository path.
dep_path, file_path = parse_util.GetDepPathAndNormalizedFilePath(
- file_path, deps)
-
- #TODO(katesonia): Enable component classifier later.
- component = ''
+ raw_file_path, deps)
# If we have the common stack frame index pattern, then use it
# since it is more reliable.
@@ -139,5 +139,5 @@ class CallStack(list):
else:
stack_frame_index = len(self)
- self.append(StackFrame(stack_frame_index, dep_path, component,
- function, file_path, crashed_line_numbers))
+ self.append(StackFrame(stack_frame_index, dep_path, function,
+ file_path, raw_file_path, crashed_line_numbers))
« no previous file with comments | « no previous file | appengine/findit/crash/classifier.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698