Actually check Rietveld for LGTMs in CheckOwners()

presubmit_support.py

Index: presubmit_support.py
diff --git a/presubmit_support.py b/presubmit_support.py
index 3af5edd87a54a806043687620a970f1d93652a20..0868a5d7bbd6f442d68204e5f7895f825bcad400 100755
--- a/presubmit_support.py
+++ b/presubmit_support.py
@@ -227,7 +227,10 @@ class InputApi(object):
       r"(|.*[\\\/])\.svn[\\\/].*",
   )
 
-  def __init__(self, change, presubmit_path, is_committing):
+  # TODO(dpranke): Update callers to pass in is_tbr, host_url, remove
+  # default arguments.
+  def __init__(self, change, presubmit_path, is_committing, is_tbr=False,
+    host_url='http://codereview.chromium.org'):
     """Builds an InputApi object.
 
     Args:
@@ -238,7 +241,9 @@ class InputApi(object):
     # Version number of the presubmit_support script.
     self.version = [int(x) for x in __version__.split('.')]
     self.change = change
+    self.host_url = host_url
     self.is_committing = is_committing
+    self.is_tbr = is_tbr
 
     # We expose various modules and functions as attributes of the input_api
     # so that presubmit scripts don't have to import them.
@@ -653,10 +658,6 @@ class Change(object):
     self._local_root = os.path.abspath(local_root)
     self.issue = issue
     self.patchset = patchset
-
-    # TODO(dpranke): implement - get from the patchset?
-    self.approvers = set()
-
     self.scm = ''
 
     # From the description text, build up a dictionary of key/value pairs