[Culprit-Finder] Refactor GitilesRepostory to make http_client required argument.

appengine/findit/crash/test/findit_for_chromecrash_test.py

Index: appengine/findit/crash/test/findit_for_chromecrash_test.py
diff --git a/appengine/findit/crash/test/findit_for_chromecrash_test.py b/appengine/findit/crash/test/findit_for_chromecrash_test.py
index e2edc1a62bbd10148c4103c10f8c4101f5912282..ab1c64121fbe43e7c279cd7cc8cbf59ab596109a 100644
--- a/appengine/findit/crash/test/findit_for_chromecrash_test.py
+++ b/appengine/findit/crash/test/findit_for_chromecrash_test.py
@@ -69,7 +69,7 @@ def _FinditForFracas():
 class FinditForChromeCrashTest(CrashTestCase):
 
   chrome_dep_fetcher = chrome_dependency_fetcher.ChromeDependencyFetcher(
-      gitiles_repository.GitilesRepository(http_client=HttpClientAppengine()))
+      gitiles_repository.GitilesRepository(HttpClientAppengine()))
 
   # TODO(wrengr): what was the purpose of this test? As written it's
   # just testing that mocking works. I'm guessing it was to check that
@@ -89,7 +89,7 @@ class FinditForChromeCrashTest(CrashTestCase):
     analysis.client_id = CrashClient.FRACAS
 
     findit_client = _FinditForChromeCrash(
-        gitiles_repository.GitilesRepository(http_client=HttpClientAppengine()))
+        gitiles_repository.GitilesRepository(HttpClientAppengine()))
     self.assertIsNone(findit_client.FindCulprit(analysis))