[Findit] Reorganizing findit_for_*.py

appengine/findit/crash/findit_for_crash.py

Index: appengine/findit/crash/findit_for_crash.py
diff --git a/appengine/findit/crash/findit_for_crash.py b/appengine/findit/crash/findit_for_crash.py
deleted file mode 100644
index 829a84496acace5f3d7d01386b9f74ae39502481..0000000000000000000000000000000000000000
--- a/appengine/findit/crash/findit_for_crash.py
+++ /dev/null
@@ -1,260 +0,0 @@
-# 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 defaultdict
-
-from common.diff import ChangeType
-from common.git_repository import GitRepository
-from common.http_client_appengine import HttpClientAppengine
-from crash import crash_util
-from crash.stacktrace import CallStack
-from crash.stacktrace import Stacktrace
-from crash.results import MatchResults
-from crash.scorers.aggregated_scorer import AggregatedScorer
-from crash.scorers.min_distance import MinDistance
-from crash.scorers.top_frame_index import TopFrameIndex
-
-
-def GetDepsInCrashStack(crash_stack, crash_deps):
-  """Gets Dependencies in crash stack."""
-  if not crash_stack:
-    return {}
-
-  stack_deps = {}
-  for frame in crash_stack:
-    if frame.dep_path:
-      stack_deps[frame.dep_path] = crash_deps[frame.dep_path]
-
-  return stack_deps
-
-
-# TODO(katesonia): Remove the repository argument after refatoring cl committed.
-def GetChangeLogsForFilesGroupedByDeps(regression_deps_rolls, stack_deps,
-                                       repository):
-  """Gets a dict containing files touched by changelogs for deps in stack_deps.
-
-  Regression ranges for each dep is determined by regression_deps_rolls.
-  Those changelogs got reverted should be returned in a ignore_cls set.
-
-  Args:
-    regression_deps_rolls (dict): Maps dep_path to DependencyRoll in
-      regression range.
-    stack_deps (dict): Represents all the dependencies shown in
-      the crash stack.
-    repository (Repository): Repository to get changelogs from.
-
-  Returns:
-    A tuple (dep_to_file_to_changelogs, ignore_cls).
-
-    dep_to_file_to_changelogs (dict): Maps dep_path to a dict mapping file path
-      to ChangeLogs that touched this file.
-    For example:
-    {
-        'src/': {
-            'a.cc': [
-                ChangeLog.FromDict({
-                    'author_name': 'test@chromium.org',
-                    'message': 'dummy',
-                    'committer_email': 'example@chromium.org',
-                    'commit_position': 175976,
-                    'author_email': 'example@chromium.org',
-                    'touched_files': [
-                        {
-                            'change_type': 'add',
-                            'new_path': 'a.cc',
-                            'old_path': 'b/a.cc'
-                        },
-                        ...
-                    ],
-                    'author_time': 'Thu Mar 31 21:24:43 2016',
-                    'committer_time': 'Thu Mar 31 21:28:39 2016',
-                    'commit_url':
-                        'https://repo.test/+/bcfd',
-                    'code_review_url': 'https://codereview.chromium.org/3281',
-                    'committer_name': 'example@chromium.org',
-                    'revision': 'bcfd',
-                    'reverted_revision': None
-                }),
-            ]
-        }
-    }
-
-    ignore_cls (set): A set of reverted revisions.
-  """
-  dep_to_file_to_changelogs = defaultdict(lambda: defaultdict(list))
-  ignore_cls = set()
-
-  for dep in stack_deps:
-    # If a dep is not in regression range, than it cannot be the dep of
-    # culprits.
-    if dep not in regression_deps_rolls:
-      continue
-
-    dep_roll = regression_deps_rolls[dep]
-
-    repository.repo_url = dep_roll.repo_url
-    changelogs = repository.GetChangeLogs(dep_roll.old_revision,
-                                          dep_roll.new_revision)
-
-    for changelog in changelogs:
-      if changelog.reverted_revision:
-        # Skip reverting cls and add reverted revisions to ignore_cls to later
-        # filter those reverted revisions.
-        ignore_cls.add(changelog.reverted_revision)
-        continue
-
-      for touched_file in changelog.touched_files:
-        if touched_file.change_type == ChangeType.DELETE:
-          continue
-
-        dep_to_file_to_changelogs[dep][touched_file.new_path].append(changelog)
-
-  return dep_to_file_to_changelogs, ignore_cls
-
-
-def GetStackInfosForFilesGroupedByDeps(stacktrace, stack_deps):
-  """Gets a dict containing all the stack information of files in stacktrace.
-
-  Only gets stack informations for files grouped by deps in stack_deps.
-
-  Args:
-    stacktrace (Stacktrace): Parsed stacktrace object.
-    stack_deps (dict): Represents all the dependencies show in
-      the crash stack.
-
-  Returns:
-    A dict, maps dep path to a dict mapping file path to a list of stack
-    inforamtion of this file. A file may occur in several frames, one stack info
-    consist of a StackFrame and the callstack priority of it.
-
-    For example:
-    {
-        'src/': {
-            'a.cc': [
-                (StackFrame(0, 'src/', '', 'func', 'a.cc', [1]), 0),
-                (StackFrame(2, 'src/', '', 'func', 'a.cc', [33]), 0),
-            ]
-        }
-    }
-  """
-  dep_to_file_to_stack_infos = defaultdict(lambda: defaultdict(list))
-
-  for callstack in stacktrace:
-    for frame in callstack:
-      # We only care about those dependencies in crash stack.
-      if frame.dep_path not in stack_deps:
-        continue
-
-      dep_to_file_to_stack_infos[frame.dep_path][frame.file_path].append((
-          frame, callstack.priority))
-
-  return dep_to_file_to_stack_infos
-
-
-# TODO(katesonia): Remove the repository argument after refatoring cl committed.
-def FindMatchResults(dep_to_file_to_changelogs,
-                     dep_to_file_to_stack_infos,
-                     stack_deps, repository,
-                     ignore_cls=None):
-  """Finds results by matching stacktrace and changelogs in regression range.
-
-  This method only applies to those crashes with regression range.
-
-  Args:
-    dep_to_file_to_changelogs (dict): Maps dep_path to a dict mapping file path
-      to ChangeLogs that touched this file.
-    dep_to_file_to_stack_infos (dict): Maps dep path to a dict mapping file path
-      to a list of stack inforamtion of this file. A file may occur in several
-      frames, one stack info consist of a StackFrame and the callstack priority
-      of it.
-    stack_deps (dict): Represents all the dependencies shown in the crash stack.
-    repository (Repository): Repository to get changelogs and blame from.
-    ignore_cls (set): Set of reverted revisions.
-
-  Returns:
-    A list of MatchResult instances with confidence and reason unset.
-  """
-  match_results = MatchResults(ignore_cls)
-
-  for dep, file_to_stack_infos in dep_to_file_to_stack_infos.iteritems():
-    file_to_changelogs = dep_to_file_to_changelogs[dep]
-    repository.repo_url = stack_deps[dep].repo_url
-
-    for crashed_file_path, stack_infos in file_to_stack_infos.iteritems():
-      for touched_file_path, changelogs in file_to_changelogs.iteritems():
-        if not crash_util.IsSameFilePath(crashed_file_path, touched_file_path):
-          continue
-
-        blame = repository.GetBlame(crashed_file_path,
-                                    stack_deps[dep].revision)
-
-        # Generate/update each result(changelog) in changelogs, blame is used
-        # to calculate distance between touched lines and crashed lines in file.
-        match_results.GenerateMatchResults(
-            crashed_file_path, dep, stack_infos, changelogs, blame)
-
-  return match_results.values()
-
-
-# TODO(katesonia): Remove the repository argument after refatoring cl committed.
-def FindItForCrash(stacktrace, regression_deps_rolls, crashed_deps, top_n,
-                   repository):
-  """Finds culprit results for crash.
-
-  Args:
-    stacktrace (Stactrace): Parsed Stactrace object.
-    regression_deps_rolls (dict): Maps dep_path to DependencyRoll in
-      regression range.
-    crashed_deps (dict of Dependencys): Represents all the dependencies of
-      crashed revision.
-    top_n (int): Top n frames of each stack to be analyzed.
-    repository (Repository): Repository to get changelogs and blame from.
-
-  Returns:
-    List of Results, sorted by confidence from highest to lowest.
-  """
-  if not regression_deps_rolls:
-    return []
-
-  # Findit will only analyze the top n frames in each callstacks.
-  stack_trace = Stacktrace([
-      CallStack(stack.priority,
-                format_type=stack.format_type,
-                language_type=stack.language_type,
-                frame_list=stack[:top_n])
-      for stack in stacktrace])
-
-  # We are only interested in the deps in crash stack (the callstack that
-  # caused the crash).
-  stack_deps = GetDepsInCrashStack(stack_trace.crash_stack, crashed_deps)
-
-  # Get dep and file to changelogs, stack_info and blame dicts.
-  dep_to_file_to_changelogs, ignore_cls = GetChangeLogsForFilesGroupedByDeps(
-      regression_deps_rolls, stack_deps, repository)
-  dep_to_file_to_stack_infos = GetStackInfosForFilesGroupedByDeps(
-      stack_trace, stack_deps)
-
-  results = FindMatchResults(dep_to_file_to_changelogs,
-                             dep_to_file_to_stack_infos,
-                             stack_deps, repository, ignore_cls)
-
-  if not results:
-    return []
-
-  aggregated_scorer = AggregatedScorer([TopFrameIndex(), MinDistance()])
-
-  # Set result.confidence, result.reasons and result.changed_files.
-  map(aggregated_scorer.Score, results)
-
-  # Filter all the 0 confidence results.
-  results = filter(lambda r: r.confidence != 0, results)
-  if not results:
-    return []
-
-  sorted_results = sorted(results, key=lambda r: -r.confidence)
-
-  if sorted_results[0].confidence > 0.999:
-    return sorted_results[:1]
-
-  return sorted_results[:3]