Actually check Rietveld for LGTMs in CheckOwners()

presubmit_support.py

 #!/usr/bin/python
 # Copyright (c) 2010 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.
 
 """Enables directory-specific presubmit checks to run at upload and/or commit.
 """
 
 __version__ = '1.3.5'
 
@@ skipping 209 matching lines @@
       r".*\bRelease[\\\/].*",
       r".*\bxcodebuild[\\\/].*",
       r".*\bsconsbuild[\\\/].*",
       # All caps files like README and LICENCE.
       r".*\b[A-Z0-9_]{2,}$",
       # SCM (can happen in dual SCM configuration). (Slightly over aggressive)
       r"(|.*[\\\/])\.git[\\\/].*",
       r"(|.*[\\\/])\.svn[\\\/].*",
   )
 
-  def __init__(self, change, presubmit_path, is_committing):
+  # TODO(dpranke): Update callers to pass in tbr, host_url, remove
+  # default arguments.
+  def __init__(self, change, presubmit_path, is_committing, tbr=False,
+    host_url='http://codereview.chromium.org'):
     """Builds an InputApi object.
 
     Args:
       change: A presubmit.Change object.
       presubmit_path: The path to the presubmit script being processed.
       is_committing: True if the change is about to be committed.
     """
     # 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.tbr = tbr
 
     # We expose various modules and functions as attributes of the input_api
     # so that presubmit scripts don't have to import them.
     self.basename = os.path.basename
     self.cPickle = cPickle
     self.cStringIO = cStringIO
     self.json = json
     self.os_path = os.path
     self.pickle = pickle
     self.marshal = marshal
@@ skipping 394 matching lines @@
 
   def __init__(self, name, description, local_root, files, issue, patchset):
     if files is None:
       files = []
     self._name = name
     self._full_description = description
     # Convert root into an absolute path.
     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
     # plus the description minus all key/value or "tag" lines.
     description_without_tags = []
     self.tags = {}
     for line in self._full_description.splitlines():
       m = self._TAG_LINE_RE.match(line)
       if m:
         self.tags[m.group('key')] = m.group('value')
@@ skipping 514 matching lines @@
                                options.commit,
                                options.verbose,
                                sys.stdout,
                                sys.stdin,
                                options.default_presubmit,
                                options.may_prompt)
 
 
 if __name__ == '__main__':
   sys.exit(Main(sys.argv))