Moving ScheduleNewAnalysis to break the cycle

appengine/findit/crash/changelist_classifier.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.
 
 import logging
 from collections import defaultdict
 
 from common import chrome_dependency_fetcher
 from crash import crash_util
 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
 from crash.stacktrace import CallStack
 from crash.stacktrace import Stacktrace
 from lib.gitiles.diff import ChangeType
 
-# TODO(wrengr): make this a namedtuple.
+# TODO(http://crbug.com/661822): convert this into a namedtuple.
 class ChangelistClassifier(object):
   def __init__(self, repository,
       top_n_frames, top_n_results=3, confidence_threshold=0.999):
     """Args:
       repository (Repository): the Git repository for getting CLs to classify.
       top_n_frames (int): how many frames of each callstack to look at.
       top_n_results (int): maximum number of results to return.
       confidence_threshold (float): In [0,1], above which we only return
         the first result.
     """
@@ skipping 272 matching lines @@
 
         blame = repository.GetBlame(touched_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(
             touched_file_path, dep, stack_infos, changelogs, blame)
 
   return match_results.values()