Actually check Rietveld for LGTMs in CheckOwners()

owners.py

 # 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.
 
 """A database of OWNERS files."""
 
 import re
 
 
 # If this is present by itself on a line, this means that everyone can review.
@@ skipping 26 matching lines @@
     """Args:
       root: the path to the root of the Repository
       open: function callback to open a text file for reading
       os_path: module/object callback with fields for 'abspath', 'dirname',
           'exists', and 'join'
     """
     self.root = root
     self.fopen = fopen
     self.os_path = os_path
 
-    # TODO: Figure out how to share the owners email addr format w/
-    # tools/commit-queue/projects.py, especially for per-repo whitelists.
+    # Pick a default email regexp to use; callers can override as desired.
     self.email_regexp = re.compile(BASIC_EMAIL_REGEXP)
 
     # Mapping of owners to the paths they own.
     self.owned_by = {EVERYONE: set()}
 
     # Mapping of paths to authorized owners.
     self.owners_for = {}
 
     # Set of paths that stop us from looking above them for owners.
     # (This is implicitly true for the root directory).
@@ skipping 96 matching lines @@
     # short combinations of owners.
     every_owner = set()
     for f in files:
       dirname = self.os_path.dirname(f)
       while dirname in self.owners_for:
         every_owner |= self.owners_for[dirname]
         if self._stop_looking(dirname):
           break
         dirname = self.os_path.dirname(dirname)
     return every_owner