[Findit] Factoring the gitiles (etc) stuff out into its own directory

appengine/findit/crash/crash_pipeline.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 copy
 import json
 import logging
 
 from common import appengine_util
 from common import constants
-from common import git_repository
 from common import pubsub_util
 from common import time_util
 from common.http_client_appengine import HttpClientAppengine
 from common.pipeline_wrapper import BasePipeline
 from common.pipeline_wrapper import pipeline
 from crash import findit_for_chromecrash
 from crash import findit_for_clusterfuzz
 from crash.type_enums import CrashClient
+from lib.gitiles import gitiles_repository
 from model import analysis_status
 from model.crash.crash_config import CrashConfig
 
 
 # TODO(http://crbug.com/659346): write complete coverage tests for this.
 def FinditForClientID(client_id): # pragma: no cover
   """Construct a Findit object from a client id string specifying the class.
 
   We cannot pass Findit objects to the various methods in
   |crash.crash_pipeline|, because they are not JSON serializable. For now,
@@ skipping 15 matching lines @@
     cls = findit_for_chromecrash.FinditForFracas
   elif client_id == CrashClient.CRACAS:
     cls = findit_for_chromecrash.FinditForCracas
   elif client_id == CrashClient.CLUSTERFUZZ:
     cls = findit_for_clusterfuzz.FinditForClusterfuzz
   else:
     raise ValueError('FinditForClientID: '
         'unknown or unsupported client %s' % client_id)
 
   return cls(
-      git_repository.GitRepository(http_client=HttpClientAppengine()),
+      gitiles_repository.GitilesRepository(http_client=HttpClientAppengine()),
       CrashWrapperPipeline)
 
 
 # Some notes about the classes below, for people who are not
 # familiar with AppEngine. The thing that really kicks everything off
 # is |CrashWrapperPipeline.run|. However, an important thing to bear in
 # mind is that whatever arguments are passed to that method will also
 # be passed to the |run| method on whatever objects it yields. Thus,
 # all the |run| methods across these different classes must have the same
 # type. In practice, we end up passing all the arguments to the
@@ skipping 107 matching lines @@
     self._crash_identifiers = crash_identifiers
     self._client_id = client_id
 
   # TODO(http://crbug.com/659346): write coverage tests.
   # Arguments number differs from overridden method - pylint: disable=W0221
   def run(self): # pragma: no cover
     run_analysis = yield CrashAnalysisPipeline(
         self._client_id, self._crash_identifiers)
     with pipeline.After(run_analysis):
       yield PublishResultPipeline(self._client_id, self._crash_identifiers)