First stab at using Rietveld wrapper object in both gcl.py and git-cl.

rietveld.py

 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 """Defines class Rietveld to easily access a rietveld instance.
 
 Security implications:
 
 The following hypothesis are made:
 - Rietveld enforces:
   - Nobody else than issue owner can upload a patch set
@@ skipping 24 matching lines @@
 # Hack out upload logging.info()
 upload.logging = logging.getLogger('upload')
 # Mac pylint choke on this line.
 upload.logging.setLevel(logging.WARNING)  # pylint: disable=E1103
 
 
 class Rietveld(object):
   """Accesses rietveld."""
   def __init__(self, url, email, password):
     self.issue = None
-    self.user = email
     self.url = url
-    self._get_creds = lambda: (email, password)
+    if email and password:
+      get_creds = lambda: (email, password)
+      self.rpc_server = upload.HttpRpcServer(
+            self.url,
+            get_creds)
+    else:
+      self.rpc_server = upload.GetRpcServer(url, email)
     self._xsrf_token = None
     self._xsrf_token_time = None
-    self.rpc_server = upload.HttpRpcServer(
-          self.url,
-          self._get_creds,
-          save_cookies=False)
 
   def xsrf_token(self):
     if (not self._xsrf_token_time or
         (time.time() - self._xsrf_token_time) > 30*60):
       self._xsrf_token_time = time.time()
       self._xsrf_token = self.get(
           '/xsrf_token',
           extra_headers={'X-Requesting-XSRF-Token': '1'})
     return self._xsrf_token
 
@@ skipping 104 matching lines @@
         ('send_mail', 'True'),
         ('no_redirect', 'True')])
 
   def set_flag(self, issue, patchset, flag, value):
     return self.post('/%s/edit_flags' % issue, [
         ('last_patchset', str(patchset)),
         ('xsrf_token', self.xsrf_token()),
         (flag, value)])
 
   def get(self, request_path, **kwargs):
-    return self._send(request_path, payload=None, **kwargs)
+    kwargs.setdefault('payload', None)
+    return self._send(request_path, **kwargs)
 
   def post(self, request_path, data, **kwargs):
     ctype, body = upload.EncodeMultipartFormData(data, [])
     return self._send(request_path, payload=body, content_type=ctype, **kwargs)
 
   def _send(self, request_path, **kwargs):
     """Sends a POST/GET to Rietveld.  Returns the response body."""
     maxtries = 5
     for retry in xrange(maxtries):
       try:
         result = self.rpc_server.Send(request_path, **kwargs)
         # Sometimes GAE returns a HTTP 200 but with HTTP 500 as the content. How
         # nice.
         return result
       except urllib2.HTTPError, e:
         if retry >= (maxtries - 1):
           raise
         if e.code not in (500, 502, 503):
           raise
       except urllib2.URLError, e:
         if retry >= (maxtries - 1):
           raise
         if not 'Name or service not known' in e.reason:
           # Usually internal GAE flakiness.
           raise
       # If reaching this line, loop again. Uses a small backoff.
       time.sleep(1+maxtries*2)
+
+  # DEPRECATED.
+  Send = get