[Findit] Add local git parsers.

appengine/findit/common/blame.py

 # Copyright 2014 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.
 
 
 class Region(object):
   """A region of some (unspecified) file at a (known) revision."""
   def __init__(self, start, count, revision,
-               author_name, author_email, author_time):
+               author_name=None, author_email=None, author_time=None):
     self.start = start
     self.count = count
     self.revision = revision
     self.author_name = author_name
     self.author_email = author_email
     self.author_time = author_time
 
   def ToDict(self):
     return {
         'start': self.start,
         'count': self.count,
         'revision': self.revision,
         'author_name': self.author_name,
         'author_email': self.author_email,
         'author_time': self.author_time
     }
 
 
 class Blame(list):
   """A list of regions for a (known) revision of a (known) file."""
-  def __init__(self, revision, path):
-    super(Blame, self).__init__()
+  def __init__(self, revision, path, regions=None):
+    super(Blame, self).__init__(regions or [])

[stgao, 2016/10/22 00:39:16]

Can we just use the AddRegion below or remove AddRegion if we want to pass the
regions in the constructor?

This doesn't look that clean to me.

[Sharu Jiang, 2016/10/25 00:36:19]

This is removed in the newest patch.

     self.revision = revision
     self.path = path
 
   def AddRegion(self, region):
     self.append(region)
 
   def ToDict(self):
     regions = []
     for region in self:
       regions.append(region.ToDict())
     return {
         'revision': self.revision,
         'path': self.path,
         'regions': regions
     }