[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..c7b3a6584497bbfaaf724ef4d88592522d1f97ae 100644
--- a/appengine/findit/common/rietveld.py
+++ b/appengine/findit/common/rietveld.py
@@ -2,18 +2,27 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+# TODO: In the new layout, this should move to the ./services or
+# ./services/waterfall_app directories, since it is only used by Waterfall.
+
+# TODO: we ought to abstract over the HTTP_CLIENT member (which is only
+# used by the Post method) by passing it to the constructor. That way
+# things are more losely coupled, improving modularity and reducing
+# fragility. In addition, for easier mocking, we may want to just have
+# the thing passed for HTTP_CLIENT to be ``callable``, rather than giving
+# a name to the method we use on that object.
+
 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)