libv8 7.3.492.27.1 → 8.4.255.0

data/vendor/depot_tools/recipes/recipe_modules/tryserver/examples/full.expected/with_wrong_patch.json

@@ -10,33 +10,12 @@
     ],
     "infra_step": true,
     "name": "git diff to analyze patch",
-    "stdout": "/path/to/tmp/",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@files@foo.cc@@@",
-      "@@@STEP_LOG_END@files@@@",
-      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@"
+      "@@@STEP_LOG_END@files@@@"
     ]
   },
   {
-    "cmd": [
-      "python",
-      "-u",
-      "import sys; sys.exit(1)"
-    ],
-    "name": "fail",
-    "~followup_annotations": [
-      "@@@STEP_TEXT@foo@@@",
-      "@@@STEP_FAILURE@@@",
-      "@@@SET_BUILD_PROPERTY@failure_hash@\"c2311ad770732eade3d2f48247abd147e40a70e7\"@@@"
-    ]
-  },
-  {
-    "failure": {
-      "failure": {
-        "step": "fail"
-      },
-      "humanReason": "Step('fail') failed with return_code 1"
-    },
     "name": "$result"
   }
 ]

data/vendor/depot_tools/recipes/recipe_modules/tryserver/examples/full.expected/with_wrong_patch_new.json

@@ -11,33 +11,12 @@
     "cwd": "[START_DIR]\\sub\\project",
     "infra_step": true,
     "name": "git diff to analyze patch",
-    "stdout": "/path/to/tmp/",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@files@sub/project/foo.cc@@@",
-      "@@@STEP_LOG_END@files@@@",
-      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@"
+      "@@@STEP_LOG_END@files@@@"
     ]
   },
   {
-    "cmd": [
-      "python",
-      "-u",
-      "import sys; sys.exit(1)"
-    ],
-    "name": "fail",
-    "~followup_annotations": [
-      "@@@STEP_TEXT@foo@@@",
-      "@@@STEP_FAILURE@@@",
-      "@@@SET_BUILD_PROPERTY@failure_hash@\"c2311ad770732eade3d2f48247abd147e40a70e7\"@@@"
-    ]
-  },
-  {
-    "failure": {
-      "failure": {
-        "step": "fail"
-      },
-      "humanReason": "Step('fail') failed with return_code 1"
-    },
     "name": "$result"
   }
 ]

data/vendor/depot_tools/recipes/recipe_modules/tryserver/examples/full.py

@@ -17,10 +17,6 @@ DEPS = [
 
 
 def RunSteps(api):
-  if api.properties.get('set_failure_hash_with_no_steps'):
-    with api.tryserver.set_failure_hash():
-      raise api.step.StepFailure('boom!')
-
   api.path['checkout'] = api.path['start_dir']
   if api.properties.get('patch_text'):
     api.step('patch_text test', [
@@ -34,7 +30,8 @@ def RunSteps(api):
     assert (api.tryserver.gerrit_change_repo_url ==
             'https://chromium.googlesource.com/chromium/src')
     assert api.tryserver.gerrit_change_fetch_ref == 'refs/changes/27/91827/1'
-    expected_target_ref = api.properties.get('target_ref', 'refs/heads/master')
+    expected_target_ref = api.properties.get(
+        'expected_target_ref', 'refs/heads/master')
     assert api.tryserver.gerrit_change_target_ref == expected_target_ref
 
   if api.tryserver.is_gerrit_issue:
@@ -46,40 +43,19 @@ def RunSteps(api):
     api.tryserver.set_subproject_tag('v8')
 
   api.tryserver.set_patch_failure_tryjob_result()
-  api.tryserver.set_do_not_retry_build()
   api.tryserver.set_compile_failure_tryjob_result()
   api.tryserver.set_test_failure_tryjob_result()
   api.tryserver.set_invalid_test_results_tryjob_result()
+  api.tryserver.set_test_timeout_tryjob_result()
+  api.tryserver.set_test_expired_tryjob_result()
 
   api.tryserver.normalize_footer_name('Cr-Commit-Position')
 
-  with api.tryserver.set_failure_hash():
-    api.python.failing_step('fail', 'foo')
-
 
 def GenTests(api):
-  description_step = api.override_step_data(
-      'git_cl description', stdout=api.raw_io.output_text('foobar'))
   # The 'test_patch_root' property used below is just so that these
   # tests can avoid using the gclient module to calculate the
   # patch root. Normal users would use gclient.get_gerrit_patch_root().
-  yield (api.test('with_git_patch') +
-         api.properties(
-             path_config='buildbot',
-             patch_storage='git',
-             patch_project='v8',
-             patch_repo_url='http://patch.url/',
-             patch_ref='johndoe#123.diff',
-             test_patch_root='v8'))
-
-  yield (api.test('with_git_patch_luci') +
-         api.properties(
-             patch_storage='git',
-             patch_project='v8',
-             patch_repo_url='http://patch.url/',
-             patch_ref='johndoe#123.diff',
-             test_patch_root='v8'))
-
   yield (api.test('with_wrong_patch') +
          api.platform('win', 32) +
          api.properties(test_patch_root=''))
@@ -99,7 +75,7 @@ def GenTests(api):
             git_repo='https://chromium.googlesource.com/chromium/src',
             change_number=91827,
             patch_set=1) +
-         api.properties(target_ref='refs/heads/experiment') +
+         api.properties(expected_target_ref='refs/heads/experiment') +
          api.tryserver.gerrit_change_target_ref('refs/heads/experiment'))
 
   yield (api.test('with_wrong_patch_new') + api.platform('win', 32) +
@@ -117,6 +93,3 @@ def GenTests(api):
              'parse description (2)',
              api.json.output({'Foo': ['bar']}))
   )
-
-  yield (api.test('set_failure_hash_with_no_steps') +
-         api.properties(set_failure_hash_with_no_steps=True))

data/vendor/depot_tools/recipes/recipe_modules/tryserver/tests/gerrit_change_fetch_ref_timeout.py

@@ -0,0 +1,29 @@
+# Copyright 2019 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 recipe_engine import post_process
+
+DEPS = [
+    'gerrit',
+    'recipe_engine/buildbucket',
+    'recipe_engine/properties',
+    'tryserver',
+]
+
+
+def RunSteps(api):
+  api.tryserver.gerrit_change_fetch_ref
+
+
+def GenTests(api):
+  yield (api.test('timeout') + api.buildbucket.try_build(
+      'chromium',
+      'linux',
+      git_repo='https://chromium.googlesource.com/chromium/src',
+      change_number=91827,
+      patch_set=1) + api.tryserver.gerrit_change_target_ref('refs/heads/master')
+         + api.override_step_data(
+             'gerrit fetch current CL info', times_out_after=1200) +
+         api.post_process(post_process.StatusException) + api.post_process(
+             post_process.DropExpectation))

data/vendor/depot_tools/recipes/recipe_modules/tryserver/tests/gerrit_change_owner.expected/basic.json

@@ -0,0 +1,5 @@
+[
+  {
+    "name": "$result"
+  }
+]

data/vendor/depot_tools/recipes/recipe_modules/tryserver/tests/gerrit_change_owner.py

@@ -0,0 +1,22 @@
+# Copyright 2020 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 recipe_engine import post_process
+
+DEPS = [
+    'recipe_engine/buildbucket',
+    'gerrit',
+    'tryserver',
+]
+
+
+def RunSteps(api):
+  api.tryserver.gerrit_change_owner
+
+
+def GenTests(api):
+  yield api.test(
+      'basic',
+      api.buildbucket.try_build()
+  )

data/vendor/depot_tools/recipes/recipe_modules/tryserver/tests/gerrit_change_target_ref.py

@@ -0,0 +1,32 @@
+# Copyright 2019 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 recipe_engine import post_process
+
+DEPS = [
+    'gerrit',
+    'recipe_engine/buildbucket',
+    'recipe_engine/properties',
+    'recipe_engine/step',
+    'tryserver',
+]
+
+
+def RunSteps(api):
+  api.step(name=api.tryserver.gerrit_change_target_ref or 'None!', cmd=None)
+
+
+def GenTests(api):
+  yield api.test(
+      'works',
+      api.buildbucket.try_build(
+          'chromium',
+          'linux',
+          git_repo='https://chromium.googlesource.com/chromium/src',
+          change_number=91827,
+          patch_set=1),
+      api.tryserver.gerrit_change_target_ref('refs/branch-heads/custom'),
+      api.post_check(
+          lambda check, steps: check('refs/branch-heads/custom' in steps)),
+      api.post_process(post_process.DropExpectation))

data/vendor/depot_tools/recipes/recipe_modules/windows_sdk/examples/full.expected/linux.json

@@ -16,7 +16,6 @@
     "name": "ninja"
   },
   {
-    "jsonResult": null,
     "name": "$result"
   }
 ]

data/vendor/depot_tools/recipes/recipe_modules/windows_sdk/examples/full.expected/mac.json

@@ -16,7 +16,6 @@
     "name": "ninja"
   },
   {
-    "jsonResult": null,
     "name": "$result"
   }
 ]

data/vendor/depot_tools/recipes/recipe_modules/windows_sdk/examples/full.expected/win.json

@@ -7,6 +7,8 @@
       "[CACHE]\\windows_sdk",
       "-ensure-file",
       "chrome_internal/third_party/sdk/windows uploaded:2018-06-13",
+      "-max-threads",
+      "0",
       "-json-output",
       "/path/to/tmp/json"
     ],
@@ -101,7 +103,6 @@
     "name": "taskkill mspdbsrv"
   },
   {
-    "jsonResult": null,
     "name": "$result"
   }
 ]

data/vendor/depot_tools/recipes/recipe_modules/windows_sdk/examples/full.py

@@ -18,9 +18,4 @@ def RunSteps(api):
 
 def GenTests(api):
   for platform in ('linux', 'mac', 'win'):
-    properties = {
-        'buildername': 'test_builder',
-    }
-    yield (api.test(platform) +
-           api.platform.name(platform) +
-           api.properties.generic(**properties))
+    yield api.test(platform) + api.platform.name(platform)

data/vendor/depot_tools/recipes/recipes.py

@@ -1,9 +1,18 @@
-#!/usr/bin/env python
-
-# Copyright 2017 The LUCI Authors. All rights reserved.
+#!/bin/sh
+# Copyright 2019 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
+# We want to run python in unbuffered mode; however shebangs on linux grab the
+# entire rest of the shebang line as a single argument, leading to errors like:
+#
+#   /usr/bin/env: 'python -u': No such file or directory
+#
+# This little shell hack is a triple-quoted noop in python, but in sh it
+# evaluates to re-exec'ing this script in unbuffered mode.
+# pylint: disable=pointless-string-statement
+''''exec python -u -- "$0" ${1+"$@"} # '''
+# vi: syntax=python
 """Bootstrap script to clone and forward to the recipe engine tool.
 
 *******************
@@ -14,34 +23,32 @@ This is a copy of https://chromium.googlesource.com/infra/luci/recipes-py/+/mast
 To fix bugs, fix in the googlesource repo then run the autoroller.
 """
 
+# pylint: disable=wrong-import-position
 import argparse
+import errno
 import json
 import logging
 import os
-import random
 import subprocess
 import sys
-import time
 import urlparse
 
 from collections import namedtuple
 
-from cStringIO import StringIO
-
 # The dependency entry for the recipe_engine in the client repo's recipes.cfg
 #
 # url (str) - the url to the engine repo we want to use.
 # revision (str) - the git revision for the engine to get.
 # branch (str) - the branch to fetch for the engine as an absolute ref (e.g.
 #   refs/heads/master)
-EngineDep = namedtuple('EngineDep',
-                       'url revision branch')
+EngineDep = namedtuple('EngineDep', 'url revision branch')
 
 
 class MalformedRecipesCfg(Exception):
+
   def __init__(self, msg, path):
-    super(MalformedRecipesCfg, self).__init__('malformed recipes.cfg: %s: %r'
-                                              % (msg, path))
+    full_message = 'malformed recipes.cfg: %s: %r' % (msg, path)
+    super(MalformedRecipesCfg, self).__init__(full_message)
 
 
 def parse(repo_root, recipes_cfg_path):
@@ -79,8 +86,8 @@ def parse(repo_root, recipes_cfg_path):
 
     if 'url' not in engine:
       raise MalformedRecipesCfg(
-        'Required field "url" in dependency "recipe_engine" not found',
-        recipes_cfg_path)
+          'Required field "url" in dependency "recipe_engine" not found',
+          recipes_cfg_path)
 
     engine.setdefault('revision', '')
     engine.setdefault('branch', 'refs/heads/master')
@@ -90,8 +97,8 @@ def parse(repo_root, recipes_cfg_path):
     if not engine['branch'].startswith('refs/'):
       engine['branch'] = 'refs/heads/' + engine['branch']
 
-    recipes_path = os.path.join(
-      repo_root, recipes_path.replace('/', os.path.sep))
+    recipes_path = os.path.join(repo_root,
+                                recipes_path.replace('/', os.path.sep))
     return EngineDep(**engine), recipes_path
   except KeyError as ex:
     raise MalformedRecipesCfg(ex.message, recipes_cfg_path)
@@ -107,6 +114,7 @@ REQUIRED_BINARIES = {GIT, VPYTHON, CIPD}
 def _is_executable(path):
   return os.path.isfile(path) and os.access(path, os.X_OK)
 
+
 # TODO: Use shutil.which once we switch to Python3.
 def _is_on_path(basename):
   for path in os.environ['PATH'].split(os.pathsep):
@@ -122,13 +130,13 @@ def _subprocess_call(argv, **kwargs):
 
 
 def _git_check_call(argv, **kwargs):
-  argv = [GIT]+argv
+  argv = [GIT] + argv
   logging.info('Running %r', argv)
   subprocess.check_call(argv, **kwargs)
 
 
 def _git_output(argv, **kwargs):
-  argv = [GIT]+argv
+  argv = [GIT] + argv
   logging.info('Running %r', argv)
   return subprocess.check_output(argv, **kwargs)
 
@@ -174,15 +182,26 @@ def checkout_engine(engine_path, repo_root, recipes_cfg_path):
       _git_check_call(['init', engine_path], stdout=NUL)
 
       try:
-        _git_check_call(['rev-parse', '--verify', '%s^{commit}' % revision],
-                        cwd=engine_path, stdout=NUL, stderr=NUL)
+        _git_check_call(['rev-parse', '--verify',
+                         '%s^{commit}' % revision],
+                        cwd=engine_path,
+                        stdout=NUL,
+                        stderr=NUL)
       except subprocess.CalledProcessError:
-        _git_check_call(['fetch', url, branch], cwd=engine_path, stdout=NUL,
+        _git_check_call(['fetch', url, branch],
+                        cwd=engine_path,
+                        stdout=NUL,
                         stderr=NUL)
 
     try:
       _git_check_call(['diff', '--quiet', revision], cwd=engine_path)
     except subprocess.CalledProcessError:
+      index_lock = os.path.join(engine_path, '.git', 'index.lock')
+      try:
+        os.remove(index_lock)
+      except OSError as exc:
+        if exc.errno != errno.EEXIST:
+          logging.warn('failed to remove %r, reset will fail: %s', index_lock, exc)
       _git_check_call(['reset', '-q', '--hard', revision], cwd=engine_path)
 
     # If the engine has refactored/moved modules we need to clean all .pyc files
@@ -206,22 +225,24 @@ def main():
   if recipes_cfg_path:
     # calculate repo_root from recipes_cfg_path
     repo_root = os.path.dirname(
-      os.path.dirname(
-        os.path.dirname(recipes_cfg_path)))
+        os.path.dirname(os.path.dirname(recipes_cfg_path)))
   else:
     # find repo_root with git and calculate recipes_cfg_path
-    repo_root = (_git_output(
-      ['rev-parse', '--show-toplevel'],
-      cwd=os.path.abspath(os.path.dirname(__file__))).strip())
+    repo_root = (
+        _git_output(['rev-parse', '--show-toplevel'],
+                    cwd=os.path.abspath(os.path.dirname(__file__))).strip())
     repo_root = os.path.abspath(repo_root)
     recipes_cfg_path = os.path.join(repo_root, 'infra', 'config', 'recipes.cfg')
     args = ['--package', recipes_cfg_path] + args
 
   engine_path = checkout_engine(engine_override, repo_root, recipes_cfg_path)
 
-  return _subprocess_call([
-      VPYTHON, '-u',
-      os.path.join(engine_path, 'recipe_engine', 'main.py')] + args)
+  try:
+    return _subprocess_call(
+        [VPYTHON, '-u',
+         os.path.join(engine_path, 'recipe_engine', 'main.py')] + args)
+  except KeyboardInterrupt:
+    return 1
 
 
 if __name__ == '__main__':