Infer CL author and reviewer list from local state if the issue has not previously been uploaded.

presubmit_support.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.8.0'
 
@@ skipping 776 matching lines @@
   # Matches key/value (or "tag") lines in changelist descriptions.
   TAG_LINE_RE = re.compile(
       '^[ \t]*(?P<key>[A-Z][A-Z_0-9]*)[ \t]*=[ \t]*(?P<value>.*?)[ \t]*$')
   scm = ''
 
   def __init__(
       self, name, description, local_root, files, issue, patchset, author):
     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
     self.author_email = author
 
-    # 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._full_description = ''
     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')
-      else:
-        description_without_tags.append(line)
-
-    # Change back to text and remove whitespace at end.
-    self._description_without_tags = (
-        '\n'.join(description_without_tags).rstrip())
+    self._description_without_tags = ''
+    self.SetDescriptionText(description)
 
     assert all(
         (isinstance(f, (list, tuple)) and len(f) == 2) for f in files), files
 
     diff_cache = self._AFFECTED_FILES.DIFF_CACHE()
     self._affected_files = [
         self._AFFECTED_FILES(path, action.strip(), self._local_root, diff_cache)
         for action, path in files
     ]
 
   def Name(self):
     """Returns the change name."""
     return self._name
 
   def DescriptionText(self):
     """Returns the user-entered changelist description, minus tags.
 
     Any line in the user-provided description starting with e.g. "FOO="
     (whitespace permitted before and around) is considered a tag line.  Such
     lines are stripped out of the description this function returns.
     """
     return self._description_without_tags
 
   def FullDescriptionText(self):
     """Returns the complete changelist description including tags."""
     return self._full_description
 
+  def SetDescriptionText(self, description):
+    """Sets the full description text (including tags) to |description|.
+    
+    Also updates the list of tags."""
+    self._full_description = description
+
+    # 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')
+      else:
+        description_without_tags.append(line)
+
+    # Change back to text and remove whitespace at end.
+    self._description_without_tags = (
+        '\n'.join(description_without_tags).rstrip())
+
   def RepositoryRoot(self):
     """Returns the repository (checkout) root directory for this change,
     as an absolute path.
     """
     return self._local_root
 
   def __getattr__(self, attr):
     """Return tags directly as attributes on the object."""
     if not re.match(r"^[A-Z_]*$", attr):
       raise AttributeError(self, attr)
@@ skipping 752 matching lines @@
   except PresubmitFailure, e:
     print >> sys.stderr, e
     print >> sys.stderr, 'Maybe your depot_tools is out of date?'
     print >> sys.stderr, 'If all fails, contact maruel@'
     return 2
 
 
 if __name__ == '__main__':
   fix_encoding.fix_encoding()
   sys.exit(Main(None))