Removing the mutation in the factories for getting dep repositories

appengine/findit/libs/gitiles/gitiles_repository.py

Index: appengine/findit/libs/gitiles/gitiles_repository.py
diff --git a/appengine/findit/libs/gitiles/gitiles_repository.py b/appengine/findit/libs/gitiles/gitiles_repository.py
index 599724c3195c934eb3ca78be2afefe04a1bed192..53ad83ed7c2c435ba6a80239696d6b31dfd93808 100644
--- a/appengine/findit/libs/gitiles/gitiles_repository.py
+++ b/appengine/findit/libs/gitiles/gitiles_repository.py
@@ -43,18 +43,10 @@ class GitilesRepository(GitRepository):
   def repo_url(self):
     return self._repo_url
 
-  @repo_url.setter
-  def repo_url(self, repo_url):
-    self._repo_url = repo_url
-
   @property
   def http_client(self):
     return self._http_client
 
-  @property
-  def identifier(self):

[Sharu Jiang, 2016/12/30 00:00:46]

This identifier is used in ``_DefaultKeyGenerator`` in cache_decorator.py, we
should keep it.

[wrengr, 2017/01/03 19:01:02]

Done.

-    return self.repo_url
-
   def _SendRequestForJsonResponse(self, url, params=None):
     if params is None:  # pragma: no cover
       params = {}