[Predator] Move time_util from common/ to lib/, split code review related part to code_review_util

appengine/findit/crash/findit.py

 # Copyright 2016 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.
 
 import copy
 import logging
 
 from google.appengine.ext import ndb
 
 from common import appengine_util
 from common import chrome_dependency_fetcher
 from common import constants
-from common import time_util
 from crash.crash_report import CrashReport
+from lib import time_util
 from model import analysis_status
 from model.crash.crash_config import CrashConfig
 
 # TODO(http://crbug.com/659346): since most of our unit tests are
 # FinditForFracas-specific, wrengr moved them to findit_for_chromecrash_test.py.
 # However, now we're missing coverage for most of this file (due to the
 # buggy way coverage is computed). Need to add a bunch of new unittests
 # to get coverage back up.
 
 # TODO: this class depends on ndb stuff, and should therefore move to
@@ skipping 204 matching lines @@
     stacktrace = self.ParseStacktrace(model)
     if stacktrace is None:
       return None
 
     return self._predator.FindCulprit(CrashReport(
         crashed_version = model.crashed_version,
         signature = model.signature,
         platform = model.platform,
         stacktrace = stacktrace,
         regression_range = model.regression_range))