[Findit] Add timestamp of CL commit and fix two bugs.

tools/findit/blame.py

 # Copyright (c) 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.
 
 from threading import Lock
 
 from common import utils
 import crash_utils
 
 
 class Blame(object):
   """Represents a blame object.
 
   The object contains blame information for one line of stack, and this
   information is shown when there are no CLs that change the crashing files.
   Attributes:
     line_content: The content of the line to find the blame for.
     component_name: The name of the component for this line.
     stack_frame_index: The stack frame index of this file.
     file_name: The name of the file.
     line_number: The line that caused a crash.
     author: The author of this line on the latest revision.
-    crash_revision: The revision that caused the crash.
     revision: The latest revision of this line before the crash revision.
+    message: The commit message for the revision.
+    time: When the revision was committed.
     url: The url of the change for the revision.
     range_start: The starting range of the regression for this component.
     range_end: The ending range of the regression.
 
   """
 
   def __init__(self, line_content, component_name, stack_frame_index,
-               file_name, line_number, author, revision, message,
+               file_name, line_number, author, revision, message, time,
                url, range_start, range_end):
     # Set all the variables from the arguments.
     self.line_content = line_content
     self.component_name = component_name
     self.stack_frame_index = stack_frame_index
     self.file = file_name
     self.line_number = line_number
     self.author = author
     self.revision = revision
     self.message = message
+    self.time = time
     self.url = url
     self.range_start = range_start
     self.range_end = range_end
 
 
 class BlameList(object):
   """Represents a list of blame objects.
 
   Thread-safe.
   """
@@ skipping 73 matching lines @@
 
     # Parse blame information.
     parsed_blame_info = repository_parser.ParseBlameInfo(
         component_path, file_path, crashed_line_number, crash_revision)
 
     # If it fails to retrieve information, do not do anything.
     if not parsed_blame_info:
       return
 
     # Create blame object from the parsed info and add it to the list.
-    (line_content, revision, author, url, message) = parsed_blame_info
+    (line_content, revision, author, url, message, time) = parsed_blame_info
     blame = Blame(line_content, component_name, stack_frame_index, file_name,
-                  crashed_line_number, author, revision, message, url,
+                  crashed_line_number, author, revision, message, time, url,
                   range_start, range_end)
 
     with self.blame_list_lock:
       self.blame_list.append(blame)
 
   def FilterAndSortBlameList(self):
     """Filters and sorts the blame list."""
     # Sort the blame list by its position in stack.
     self.blame_list.sort(key=lambda blame: blame.stack_frame_index)
 
     filtered_blame_list = []
 
     for blame in self.blame_list:
       # If regression information is available, check if it needs to be
       # filtered.
       if blame.range_start and blame.range_end:
 
         # Discards results that are after the end of regression.
         if not utils.IsGitHash(blame.revision) and (
             int(blame.range_end) <= int(blame.revision)):
           continue
 
       filtered_blame_list.append(blame)
 
     self.blame_list = filtered_blame_list