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

appengine/findit/common/rietveld.py

Index: appengine/findit/common/rietveld.py
diff --git a/appengine/findit/common/rietveld.py b/appengine/findit/common/rietveld.py
index fab452cbd107029c3f3be7d7ac718975a4d3200a..d4b53abd12e756466724dd9c063ed555f21742c4 100644
--- a/appengine/findit/common/rietveld.py
+++ b/appengine/findit/common/rietveld.py
@@ -2,18 +2,28 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+# TODO(wrengr): this file is only used by waterfall. We should either
+# move it to that directory, or else factor out the dependency on
+# HttpClientAppengine by abstracting over the HTTP_CLIENT member. The
+# only thing we use/need HTTP_CLIENT for is to call the Post method,
+# which could just as well be rewritten to use the __call__ method
+# instead; hence, we could just pass an appropriate callable object to
+# the constructor (incidentally allowing multiple Rietveld objects to
+# share the same HTTP_CLIENT, if desired) rather than allocating our own
+# one automatically. Does rietveld itself actually require appengine?
+# Or is that just an incidental circumstance about our particular use
+# of it?

[stgao, 2016/09/21 23:03:25]

This should go to waterfall/infra/ as in the new code layout I sent you.

I'm against passing a callable object. Currently only Post method happens to be
in use, because the two APIs we interact with is with post. There are other APIs
with get. Thus, I'd pass the http_client instance over.

Currently, this only works on findit-for-me appengine because of authentication
using app identity.

 import logging
 import re
 import urlparse
 
-from common.codereview import CodeReview
 from common.http_client_appengine import HttpClientAppengine
 
 
 _RIETVELD_ISSUE_NUMBER_RE = re.compile('^/(\d+)/?.*')
 
 
-class Rietveld(CodeReview):
+class Rietveld(object):
   """The implementation of CodeReview interface for Rietveld."""
   HTTP_CLIENT = HttpClientAppengine(follow_redirects=False)