[Predator] Add ``Project`` class and ``ClassifySuspect`` method to project and component classifier

appengine/findit/crash/project.py

+# Copyright 2016 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.
+
+from collections import namedtuple
+import re
+
+from libs.gitiles.diff import ChangeType
+
+
+# TODO(http://crbug.com/659346): write the coverage tests.

[Martin Barbella, 2017/01/26 23:59:48]

Is there anything still blocking it? Would be nice to see this addressed.

[Sharu Jiang, 2017/01/27 03:20:42]

Forgot to add it...
Done

+class Project(namedtuple('Project',
+    ['name', 'path_regexs', 'function_regexs',
+     'host_directories'])): # pragma: no cover
+  """A representation of a "project".
+
+  For example: 'android_os', 'clank' or 'chromium'. Notably, a project knows
+  how to identify itself. Hence, given a stack frame, file path or dependency
+  path or whatever, we ask the ``Project``  whether it matches that frame,
+  CL, etc."""
+  __slots__ = ()
+
+  def __new__(cls, name, path_regexs=None,
+              function_regexs=None, host_directories=None):
+    path_regexs = [re.compile(path_regex) for path_regex in
+                   path_regexs] if path_regexs else None
+    function_regexs = [re.compile(function_regex) for function_regex in
+                       function_regexs] if function_regexs else None
+
+    return super(cls, Project).__new__(
+        cls, name, path_regexs, function_regexs, host_directories)
+
+  def MatchesStackFrame(self, frame):
+    """Returns true if this project matches the frame."""
+    for path_regex in self.path_regexs or []:
+      if path_regex.match(frame.raw_file_path):
+        return True
+
+    for function_regex in self.function_regexs or []:
+      if function_regex.match(frame.function):
+        return True
+
+    if not self.host_directories:
+      return False
+
+    for host_directory in self.host_directories:
+      if frame.dep_path.startswith(host_directory):
+        return True
+
+    return False
+
+  def MatchesTouchedFile(self, dep_path, touched_file):
+    """Returns true if this project matches file path."""
+    if touched_file.change_type == ChangeType.DELETE:
+      path = touched_file.old_path
+    else:
+      path = touched_file.new_path
+
+    path = dep_path + path
+    for path_regex in self.path_regexs or []:
+      if path_regex.match(path):
+        return True
+
+    for host_directory in self.host_directories or []:
+      if dep_path.startswith(host_directory):
+        return True
+
+    return False
+
+  def GetName(self, dep_path):
+    """Returns the project name from a dep path."""
+    if 'chromium' not in self.name:
+      return self.name
+
+    if not dep_path:
+      return None
+
+    for host_directory in self.host_directories or []:
+      if dep_path.startswith(host_directory):
+        path = dep_path[len(host_directory):]
+        if not path:
+          return self.name
+
+        return '%s-%s' % (self.name, path.split('/')[0].lower())
+
+    # Unknown path, return the whole path as project name.
+    return '%s-%s' % (self.name, '_'.join(dep_path.split('/')))