Actually check Rietveld for LGTMs in CheckOwners()

PRESUBMIT.py

Index: PRESUBMIT.py
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 8ea1fd9cf81445a293b9f7ccc3e1aa6573eb9e27..34aac70f7aa355c48e79332342e1426bca8f3c45 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -20,6 +20,10 @@ UNIT_TESTS = [
   'tests.watchlists_unittest',
 ]
 
+CODEREVIEW_SERVER = 'http://codereview.chromium.org'
+
+OWNERS_EMAIL_REGEXP = r'^[\w\-\+\%\.]+\@[\w\-\+\%\.]+$'
+
 def CommonChecks(input_api, output_api):
   output = []
   # Verify that LocalPath() is local, e.g.:
@@ -31,9 +35,13 @@ def CommonChecks(input_api, output_api):
       # Return right away because it needs to be fixed first.
       return output
 
-  output.extend(input_api.canned_checks.CheckOwners(
-      input_api,
-      output_api))
+  # TODO(dpranke): uncomment and enable :).
+  #
+  # output.extend(input_api.canned_checks.CheckOwners(
+  #     input_api,
+  #     output_api,
+  #     CODEREVIEW_SERVER,

[M-A Ruel, 2011/03/10 13:37:54]

I'm not sure there, git-cl and gcl already knows better what's the review url.
I'd prefer to not copy that string in many places.

+  #     EMAIL_REGEXP))
 
   output.extend(input_api.canned_checks.RunPythonUnitTests(
       input_api,