Removing the mutation in the factories for getting dep repositories

appengine/findit/crash/crash_pipeline.py

Index: appengine/findit/crash/crash_pipeline.py
diff --git a/appengine/findit/crash/crash_pipeline.py b/appengine/findit/crash/crash_pipeline.py
index 31355eac18df05dc5f820e9a71906f5f0f4ba0a2..7966d0c5c4edc4444f51401ba311f4b49ab8ef38 100644
--- a/appengine/findit/crash/crash_pipeline.py
+++ b/appengine/findit/crash/crash_pipeline.py
@@ -22,7 +22,7 @@ from model import analysis_status
 
 
 # TODO(http://crbug.com/659346): write complete coverage tests for this.
-def FinditForClientID(client_id, repository): # pragma: no cover
+def FinditForClientID(client_id, get_repository): # 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
@@ -52,7 +52,7 @@ def FinditForClientID(client_id, repository): # pragma: no cover
     raise ValueError('FinditForClientID: '
         'unknown or unsupported client %s' % client_id)
 
-  return cls(repository)
+  return cls(get_repository)
 
 
 # Some notes about the classes below, for people who are not familiar
@@ -93,7 +93,7 @@ class CrashBasePipeline(BasePipeline):
     self._crash_identifiers = crash_identifiers
     self._findit = FinditForClientID(
         client_id,
-        CachedGitilesRepository(HttpClientAppengine()))
+        CachedGitilesRepository.Factory(HttpClientAppengine()))
 
   @property
   def client_id(self): # pragma: no cover