[Predator] Generate all changelogs in regression ranges instead of only matched changelogs

appengine/findit/crash/suspect.py

Index: appengine/findit/crash/suspect.py
diff --git a/appengine/findit/crash/suspect.py b/appengine/findit/crash/suspect.py
index 567f2e9a11a4e2e4696107fd91fa2ba72d897aca..2efd426d1e8fd0aa33753af02da146891dd9f3a3 100644
--- a/appengine/findit/crash/suspect.py
+++ b/appengine/findit/crash/suspect.py
@@ -7,32 +7,6 @@ from collections import namedtuple
 from libs.gitiles.commit_util import DistanceBetweenLineRanges
 
 
-# TODO(wrengr): we should change things to use integers with None as
-# \"infinity\", rather than using floats.
-# TODO(http://crbug.com/644476): this class needs a better name.
-class AnalysisInfo(namedtuple('AnalysisInfo',
-    ['min_distance', 'min_distance_frame'])):
-  __slots__ = ()
-
-  def __str__(self): # pragma: no cover
-    return ('%s(min_distance = %d, min_distance_frame = %s)'
-        % (self.__class__.__name__, self.min_distance, self.min_distance_frame))
-
-
-# TODO(wrengr): it's not clear why the ``priority`` is stored at all,
-# given that every use in this file discards it. ``Result.file_to_stack_infos``
-# should just store pointers directly to the frames themselves rather
-# than needing this intermediate object.
-# TODO(http://crbug.com/644476): this class needs a better name.
-class StackInfo(namedtuple('StackInfo', ['frame', 'priority'])):
-  """Pair of a frame and the ``priority`` of the ``CallStack`` it came from."""
-  __slots__ = ()
-
-  def __str__(self): # pragma: no cover
-    return ('%s(frame = %s, priority = %f)'
-        % (self.__class__.__name__, self.frame, self.priority))
-
-
 # TODO(wrengr): maybe break this into separate unanalyzed suspect,
 # and analyzed suspect; so we can distinguish the input to
 # ``ChangelistClassifier`` from the output of it (which will amend each
@@ -58,14 +32,6 @@ class Suspect(object):
     self.reasons = reasons
     self.changed_files = changed_files
 
-    # TODO(katesonia): These file_to_* should be deprecated once we deprecate
-    # the scorer-based changelist_classifier.
-    # TODO(wrengr): (a) make these two fields private/readonly
-    # TODO(wrengr): (b) zip them together.
-    # TODO(wrengr): (c) move them to the relevant features instead.
-    self.file_to_stack_infos = {}
-    self.file_to_analysis_info = {}
-
   def ToDict(self):
     return {
         'url': self.changelog.commit_url,
@@ -79,108 +45,8 @@ class Suspect(object):
         'confidence': self.confidence,
     }
 
-  # TODO(katesonia): This is unusable for logging because in all the
-  # cases that need logging it returns the empty string! We should print
-  # this out in a more useful way (e.g., how CrashConfig is printed)
-  # so that callers don't have to use ``str(result.ToDict())`` instead. If
-  # we want a method that does what this one does, we should give it a
-  # different name that indicates what it's actually printing out.
   def ToString(self):
-    if not self.file_to_stack_infos:
-      return ''
-
-    lines = []
-    for file_path, stack_infos in self.file_to_stack_infos.iteritems():
-      line_parts = []
-      for frame, _ in stack_infos:
-        line_parts.append('frame #%d' % frame.index)
-
-      lines.append('Changed file %s crashed in %s' % (
-          file_path, ', '.join(line_parts)))
-
-    return '\n'.join(lines)
+    return str(self.ToDict())
 
   def __str__(self):
     return self.ToString()
-
-
-  def _UpdateSuspect(self, file_path, stack_infos, blame):
-    """Updates the ``Suspect`` with file path and its stack_infos and blame.
-
-    When a file_path is found both shown in stacktrace and touched by
-    the revision of this result, update result with the information of
-    this file.
-
-    Inserts the file path and its stack infos, and updates the min distance
-    if less distance is found between touched lines of this result and
-    crashed lines in the file path.
-
-    Args:
-      file_path (str): File path of the crashed file.
-      stack_infos (list of StackInfo): List of the frames of this file
-        together with their callstack priorities.
-      blame (Blame): Blame oject of this file.
-    """
-    self.file_to_stack_infos[file_path] = stack_infos
-
-    if not blame:
-      return
-
-    min_distance = float('inf')
-    min_distance_frame = stack_infos[0].frame
-    for region in blame:
-      if region.revision != self.changelog.revision:
-        continue
-
-      region_start = region.start
-      region_end = region_start + region.count - 1
-      for frame, _ in stack_infos:
-        frame_start = frame.crashed_line_numbers[0]
-        frame_end = frame.crashed_line_numbers[-1]
-        distance = DistanceBetweenLineRanges((frame_start, frame_end),
-                                             (region_start, region_end))
-        if distance < min_distance:
-          min_distance = distance
-          min_distance_frame = frame
-
-    self.file_to_analysis_info[file_path] = AnalysisInfo(
-        min_distance = min_distance,
-        min_distance_frame = min_distance_frame,
-    )
-
-
-class SuspectMap(dict):
-  """A map from revisions to the ``Suspect`` object for that revision."""
-
-  def __init__(self, ignore_cls=None):
-    super(SuspectMap, self).__init__()
-    self._ignore_cls = ignore_cls
-
-  def GenerateSuspects(self, file_path, dep_path, stack_infos, changelogs,
-      blame):
-    """Compute suspects from a list of CLs, and store them in this map.
-
-    Suspects are generated based on newly found file path, its stack_infos,
-    and all the changelogs that touched this file in the dep in regression
-    ranges, those reverted changelogs should be ignored.
-
-    Args:
-      file_path (str): File path of the crashed file.
-      dep_path (str): Path of the dependency of the file.
-      stack_infos (list): List of stack_info dicts, represents frames of this
-        file and the callstack priorities of those frames.
-      changelogs (list): List of Changelog objects in the dep in regression
-        range which touched the file.
-      blame (Blame): Blame of the file.
-    """
-    for changelog in changelogs:
-      if self._ignore_cls and changelog.revision in self._ignore_cls:
-        continue
-
-      try:
-        suspect = self[changelog.revision]
-      except KeyError:
-        suspect = Suspect(changelog, dep_path)
-        self[changelog.revision] = suspect
-
-      suspect._UpdateSuspect(file_path, stack_infos, blame)