[Predator] Run predator.

appengine/findit/crash/changelist_classifier.py

Index: appengine/findit/crash/changelist_classifier.py
diff --git a/appengine/findit/crash/changelist_classifier.py b/appengine/findit/crash/changelist_classifier.py
index 1272025e4041fb4c417a241bcb8256ed5d8992f7..7dab4cf51e7f22ff2f4862177eefd60212cf1fd6 100644
--- a/appengine/findit/crash/changelist_classifier.py
+++ b/appengine/findit/crash/changelist_classifier.py
@@ -292,13 +292,13 @@ def FindMatchResults(dep_to_file_to_changelogs,
 
   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
 
+        repository.repo_url = stack_deps[dep].repo_url

[stgao, 2016/11/17 05:03:05]

Why we want this to be in the inner for loop? Does stack_deps[dep].repo_url
change at all?

[Sharu Jiang, 2016/11/17 09:03:15]

Since for many cases, the repo_url doesn't need to be changed if no matches are
found, especially for local_git_repository, changing repo_url is more expensive.

         blame = repository.GetBlame(touched_file_path,
                                     stack_deps[dep].revision)