[Findit] Fetch DEPS from buildspec/ instead of trunk for chrome official builds.

appengine/findit/common/chromium_deps.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.
 
+import base64
+import re
+
+from common import auth_util
+from common import dependency
+from common import deps_parser
 from common import git_repository
 from common import http_client_appengine
-from common import dependency
-from common import deps_parser
-
 
 _CHROMIUM_ROOT_DIR = 'src/'
 _CHROMIUM_REPO_MASTER = 'https://chromium.googlesource.com/chromium/src.git'
 
+_CHROME_VERSION_PATTERN = re.compile(r'^\d+\.\d+\.\d+\.\d+$')
+
+_BUILDSPEC_REPO = ('https://chrome-internal.googlesource.com/chrome/tools/'
+                   'buildspec.git/')
+
+
+def IsChromeVersion(revision):
+  """Determines if a revision is a chrome version."""
+  if _CHROME_VERSION_PATTERN.match(revision):
+    return True
+
+  return False
+
 
 class DEPSDownloader(deps_parser.DEPSLoader):
   """Downloads DEPS from remote Git repo."""
+
   def __init__(self, check_deps_git_first=False):
     """
     Args:
       check_deps_git_first (bool): If True, use .DEPS.git instead of DEPS.
     """
     self.check_deps_git_first = check_deps_git_first
 
   def Load(self, repo_url, revision, deps_file):
     http_client = http_client_appengine.HttpClientAppengine()
+    content = None
+
+    if IsChromeVersion(revision):
+      repo = git_repository.GitRepository(_BUILDSPEC_REPO, http_client)

[stgao, 2016/05/05 21:02:47]

Can we handled both in the same way?

In abstract, both are to retrieve the DEPS file at the given deps_file path from
the given Gitiles repo at the given revision.

[Sharu Jiang, 2016/05/05 23:03:45]

Done.

+      url = '%s/+/master/releases/%s/%s' % (_BUILDSPEC_REPO, revision,

[stgao, 2016/05/05 21:02:46]

Why can't we use the repo_url passed in here?

[Sharu Jiang, 2016/05/05 23:03:45]

Done.

+                                            deps_file)
+      # The buildspec/ is a chrome internal repo, add header to verify the
+      # identity of the appengine app based on the service account.
+      headers = {'Authorization': 'Bearer ' + auth_util.GetAuthToken()}
+      content = repo._SendRequestForTextResponse(url, headers=headers)

[stgao, 2016/05/05 21:02:47]

Class-internal functions like this are not supposed to be used outside the
class.

Is there other cleaner solution?

[Sharu Jiang, 2016/05/05 23:03:45]

Done.

+
+      if content is None:
+        raise Exception(
+            'Failed to pull %s file from buildspec/ for version %s.' % (
+                deps_file, revision))
+
+      return content
+
     repo = git_repository.GitRepository(repo_url, http_client)
 
-    content = None
-
     if self.check_deps_git_first and deps_file == 'DEPS':
       # When the given deps_file is "DEPS" and .DEPS.git should be checked
       # first, it's because before migration from SVN to Git, .DEPS.git contains
       # dependencies hosted in Git while DEPS contains those in SVN.
       # If .DEPS.git is not found, fallback to the given deps_file. Assume it is
       # a commit after migration from SVN to Git.
       content = repo.GetSource('.DEPS.git', revision)
 
     if content is None:
       content = repo.GetSource(deps_file, revision)
 
     if content is None:
       raise Exception(
           'Failed to pull %s file from %s, at revision %s.' % (
               deps_file, repo_url, revision))
 
     return content
 
 
 def GetChromeDependency(revision, platform, check_deps_git_first=False):
   """Returns all dependencies of Chrome as a dict for the given revision and OS.
 
   Args:
-    revision (str): The revision of a Chrome build.
+    revision (str): The revision of a Chrome build, it can be a githash or a
+      chrome version for a official build.
     platform (str): The target platform of the Chrome build, should be one of
-        'win', 'ios', 'mac', 'unix', 'android', or 'all'.
+      'win', 'ios', 'mac', 'unix', 'android', or 'all'.
     check_deps_git_first (bool): If True, use .DEPS.git instead of DEPS.
 
   Returns:
     A map from dependency path to the dependency info.
   """
   root_dep = dependency.Dependency(
       _CHROMIUM_ROOT_DIR, _CHROMIUM_REPO_MASTER, revision, 'DEPS')
 
-  deps_parser.UpdateDependencyTree(
-      root_dep, [platform], DEPSDownloader(check_deps_git_first))
+  deps_parser.UpdateDependencyTree(root_dep, [platform],
+                                   DEPSDownloader(check_deps_git_first))
+
+  if IsChromeVersion(root_dep.revision):
+    http_client = http_client_appengine.HttpClientAppengine()
+    repo = git_repository.GitRepository(_CHROMIUM_REPO_MASTER, http_client)
+    # Convert chrome version to githash revision to be consistent.
+    root_dep.revision = repo.GetRevisionForChromeVersion(root_dep.revision)
 
   dependencies = {}
 
   # Flatten the dependency tree into a one-level dict.
   def FlattenDepTree(dep):
     dependencies[dep.path] = dep
     for child in dep.children.values():
       FlattenDepTree(child)
 
   FlattenDepTree(root_dep)
 
   return dependencies
 
 
 def GetChromiumDEPSRolls(old_cr_revision, new_cr_revision, platform,
-                         check_deps_git_first=False):
+                         check_deps_git_first=False, skip_chromium_roll=True):
   """Returns a list of dependency rolls between the given Chromium revisions.
 
   Args:
-    old_cr_revision (str): The Git commit hash for the old Chromium revision.
-    new_cr_revision (str): The Git commit hash for the new Chromium revision.
+    old_cr_revision (str): The old Chromium revision, it can be a githash or a
+      chrome version for a official build.
+    new_cr_revision (str): The new Chromium revision, it can be a githash or a
+      chrome version for a official build.
     platform (str): The target OS platform of the Chrome or test binary.
     check_deps_git_first (bool): If True, use .DEPS.git instead of DEPS.
+    skip_chromium_roll (bool): If False, chromium roll will be contained in
+      the return.
+
+  Returns:
+    A list of DependencyRoll objects in the revision range.
   """
   old_deps = GetChromeDependency(
       old_cr_revision, platform, check_deps_git_first)
   new_deps = GetChromeDependency(
       new_cr_revision, platform, check_deps_git_first)
 
   rolls = []
 
   for path, new_dep in new_deps.iteritems():
-    if path == _CHROMIUM_ROOT_DIR:  # Skip the root dependency -- chromium.
+    if skip_chromium_roll and path == _CHROMIUM_ROOT_DIR:
       continue
 
     old_revision = None
     if path in old_deps:
       old_revision = old_deps[path].revision
 
     if old_revision != new_dep.revision:
       rolls.append(
           dependency.DependencyRoll(
               path, new_dep.repo_url, old_revision, new_dep.revision))
 
   for path, old_dep in old_deps.iteritems():
     if path not in new_deps:
       rolls.append(
           dependency.DependencyRoll(
               path, old_dep.repo_url, old_dep.revision, None))
 
   return rolls
 
 
 def GetDEPSRollsDict(old_cr_revision, new_cr_revision, platform):
-  """Gets dep_path to DependencyRoll dictionary for deps in
-  (old_cr_revision, new_cr_revision].
+  """Gets dep_path to DependencyRoll dictionary for deps between revisions.
 
   Args:
-    old_cr_revision (str): The Git commit hash for the old Chromium revision.
-    new_cr_revision (str): The Git commit hash for the new Chromium revision.
+    old_cr_revision (str): The old Chromium revision, it can be a githash or a
+      chrome version for a official build.
+    new_cr_revision (str): The new Chromium revision, it can be a githash or a
+      chrome version for a official build.
     platform (str): The target OS platform of the Chrome or test binary.
 
   Returns:
-    A dict, mapping dep path to DependencyRoll.
+    A dict, mapping dep path to its DependencyRoll.
   """
-  deps_rolls = GetChromiumDEPSRolls(old_cr_revision, new_cr_revision, platform)
-  # Add chromium as dependency roll.
-  deps_rolls.append(dependency.DependencyRoll(
-      _CHROMIUM_ROOT_DIR, _CHROMIUM_REPO_MASTER,
-      old_cr_revision, new_cr_revision))
+  deps_rolls = GetChromiumDEPSRolls(old_cr_revision, new_cr_revision, platform,
+                                    skip_chromium_roll=False)
 
   deps_rolls_dict = {}
   for dep_roll in deps_rolls:
     deps_rolls_dict[dep_roll.path] = dep_roll
 
   return deps_rolls_dict