Add a presubmit test that asserts TODOs have owners....

presubmit_canned_checks.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.
 
 """Generic presubmit checks that can be reused by other presubmit checks."""
 
 ### Description checks
 
 def CheckChangeHasTestField(input_api, output_api):
   """Requires that the changelist have a TEST= field."""
@@ skipping 216 matching lines @@
   tabs = []
   for f, line_num, line in input_api.RightHandSideLines(filter_more):
     if '\t' in line:
       tabs.append('%s, line %s' % (f.LocalPath(), line_num))
   if tabs:
     return [output_api.PresubmitPromptWarning('Found a tab character in:',
                                               long_text='\n'.join(tabs))]
   return []
 
 
+def CheckChangeTodoHasOwner(input_api, output_api, source_file_filter=None):
+  """Checks that the user didn't add TODO(name) without an owner."""
+
+  unowned_todo = input_api.re.compile('TO' + 'DO[^(]');
+  for f, line_num, line in input_api.RightHandSideLines(source_file_filter):
+    if unowned_todo.search(line):
+      text = ('Found TO' + 'DO with no owner in %s, line %s' %
+              (f.LocalPath(), line_num))
+      return [output_api.PresubmitPromptWarning(text)]
+  return []
+
+
 def CheckChangeHasNoStrayWhitespace(input_api, output_api,
                                     source_file_filter=None):
   """Checks that there is no stray whitespace at source lines end."""
   errors = []
   for f, line_num, line in input_api.RightHandSideLines(source_file_filter):
     if line.rstrip() != line:
       errors.append('%s, line %s' % (f.LocalPath(), line_num))
   if errors:
     return [output_api.PresubmitPromptWarning(
         'Found line ending with white spaces in:',
@@ skipping 358 matching lines @@
     pending_builds_len = len(builder.get('pending_builds', []))
     if pending_builds_len > max_pendings:
       out.append('%s has %d build(s) pending' %
                   (builder_name, pending_builds_len))
   if out:
     return [output_api.PresubmitPromptWarning(
         'Build(s) pending. It is suggested to wait that no more than %d '
             'builds are pending.' % max_pendings,
         long_text='\n'.join(out))]
   return []