libv8 6.7.288.46.1 → 7.3.492.27.0beta1

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

@@ -0,0 +1,147 @@
+[
+  {
+    "cmd": [
+      "python",
+      "-u",
+      "RECIPE_REPO[depot_tools]/gerrit_client.py",
+      "changes",
+      "--host",
+      "https://chromium-review.googlesource.com",
+      "--json_file",
+      "/path/to/tmp/json",
+      "--limit",
+      "1",
+      "-p",
+      "change=91827",
+      "-o",
+      "ALL_REVISIONS",
+      "-o",
+      "DOWNLOAD_COMMANDS"
+    ],
+    "env": {
+      "PATH": "<PATH>:RECIPE_REPO[depot_tools]"
+    },
+    "infra_step": true,
+    "name": "gerrit fetch current CL info",
+    "~followup_annotations": [
+      "@@@STEP_LOG_LINE@json.output@[@@@",
+      "@@@STEP_LOG_LINE@json.output@  {@@@",
+      "@@@STEP_LOG_LINE@json.output@    \"branch\": \"refs/heads/experiment\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"revisions\": {@@@",
+      "@@@STEP_LOG_LINE@json.output@      \"184ebe53805e102605d11f6b143486d15c23a09c\": {@@@",
+      "@@@STEP_LOG_LINE@json.output@        \"_number\": \"1\", @@@",
+      "@@@STEP_LOG_LINE@json.output@        \"ref\": \"refs/changes/27/91827/1\"@@@",
+      "@@@STEP_LOG_LINE@json.output@      }@@@",
+      "@@@STEP_LOG_LINE@json.output@    }@@@",
+      "@@@STEP_LOG_LINE@json.output@  }@@@",
+      "@@@STEP_LOG_LINE@json.output@]@@@",
+      "@@@STEP_LOG_END@json.output@@@"
+    ]
+  },
+  {
+    "cmd": [
+      "python",
+      "-u",
+      "RECIPE_REPO[depot_tools]/gerrit_client.py",
+      "changes",
+      "--host",
+      "https://chromium-review.googlesource.com",
+      "--json_file",
+      "/path/to/tmp/json",
+      "--limit",
+      "1",
+      "-p",
+      "change=91827",
+      "-o",
+      "ALL_REVISIONS",
+      "-o",
+      "ALL_COMMITS"
+    ],
+    "env": {
+      "PATH": "<PATH>:RECIPE_REPO[depot_tools]"
+    },
+    "infra_step": true,
+    "name": "gerrit changes",
+    "~followup_annotations": [
+      "@@@STEP_LOG_LINE@json.output@[@@@",
+      "@@@STEP_LOG_LINE@json.output@  {@@@",
+      "@@@STEP_LOG_LINE@json.output@    \"_number\": \"91827\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"branch\": \"master\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"change_id\": \"Ideadbeef\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"created\": \"2017-01-30 13:11:20.000000000\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"has_review_started\": false, @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"project\": \"chromium/src\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"revisions\": {@@@",
+      "@@@STEP_LOG_LINE@json.output@      \"184ebe53805e102605d11f6b143486d15c23a09c\": {@@@",
+      "@@@STEP_LOG_LINE@json.output@        \"_number\": \"1\", @@@",
+      "@@@STEP_LOG_LINE@json.output@        \"commit\": {@@@",
+      "@@@STEP_LOG_LINE@json.output@          \"message\": \"Change commit message\"@@@",
+      "@@@STEP_LOG_LINE@json.output@        }@@@",
+      "@@@STEP_LOG_LINE@json.output@      }@@@",
+      "@@@STEP_LOG_LINE@json.output@    }, @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"status\": \"NEW\", @@@",
+      "@@@STEP_LOG_LINE@json.output@    \"subject\": \"Change title\"@@@",
+      "@@@STEP_LOG_LINE@json.output@  }@@@",
+      "@@@STEP_LOG_LINE@json.output@]@@@",
+      "@@@STEP_LOG_END@json.output@@@"
+    ]
+  },
+  {
+    "cmd": [
+      "python",
+      "-u",
+      "RECIPE_REPO[depot_tools]/git_footers.py",
+      "--json",
+      "/path/to/tmp/json"
+    ],
+    "name": "parse description",
+    "stdin": "Change commit message",
+    "~followup_annotations": [
+      "@@@STEP_LOG_END@json.output (invalid)@@@",
+      "@@@STEP_LOG_LINE@json.output (exception)@No JSON object could be decoded@@@",
+      "@@@STEP_LOG_END@json.output (exception)@@@"
+    ]
+  },
+  {
+    "cmd": [
+      "git",
+      "-c",
+      "core.quotePath=false",
+      "diff",
+      "--cached",
+      "--name-only"
+    ],
+    "infra_step": true,
+    "name": "git diff to analyze patch",
+    "stdout": "/path/to/tmp/",
+    "~followup_annotations": [
+      "@@@STEP_LOG_LINE@files@None/foo.cc@@@",
+      "@@@STEP_LOG_END@files@@@",
+      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@",
+      "@@@SET_BUILD_PROPERTY@failure_type@\"INVALID_TEST_RESULTS\"@@@",
+      "@@@SET_BUILD_PROPERTY@subproject_tag@\"v8\"@@@"
+    ]
+  },
+  {
+    "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_git_patch.json

@@ -15,6 +15,7 @@
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@files@v8/foo.cc@@@",
       "@@@STEP_LOG_END@files@@@",
+      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@",
       "@@@SET_BUILD_PROPERTY@failure_type@\"INVALID_TEST_RESULTS\"@@@",
       "@@@SET_BUILD_PROPERTY@subproject_tag@\"v8\"@@@"
     ]
@@ -27,16 +28,18 @@
     ],
     "name": "fail",
     "~followup_annotations": [
-      "step returned non-zero exit code: 1",
       "@@@STEP_TEXT@foo@@@",
       "@@@STEP_FAILURE@@@",
       "@@@SET_BUILD_PROPERTY@failure_hash@\"c2311ad770732eade3d2f48247abd147e40a70e7\"@@@"
     ]
   },
   {
-    "name": "$result",
-    "reason": "Step('fail') failed with return_code 1",
-    "recipe_result": null,
-    "status_code": 1
+    "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_git_patch_luci.json

@@ -15,6 +15,7 @@
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@files@v8/foo.cc@@@",
       "@@@STEP_LOG_END@files@@@",
+      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@",
       "@@@SET_BUILD_PROPERTY@failure_type@\"INVALID_TEST_RESULTS\"@@@",
       "@@@SET_BUILD_PROPERTY@subproject_tag@\"v8\"@@@"
     ]
@@ -27,16 +28,18 @@
     ],
     "name": "fail",
     "~followup_annotations": [
-      "step returned non-zero exit code: 1",
       "@@@STEP_TEXT@foo@@@",
       "@@@STEP_FAILURE@@@",
       "@@@SET_BUILD_PROPERTY@failure_hash@\"c2311ad770732eade3d2f48247abd147e40a70e7\"@@@"
     ]
   },
   {
-    "name": "$result",
-    "reason": "Step('fail') failed with return_code 1",
-    "recipe_result": null,
-    "status_code": 1
+    "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.json

@@ -13,7 +13,8 @@
     "stdout": "/path/to/tmp/",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@files@foo.cc@@@",
-      "@@@STEP_LOG_END@files@@@"
+      "@@@STEP_LOG_END@files@@@",
+      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@"
     ]
   },
   {
@@ -24,16 +25,18 @@
     ],
     "name": "fail",
     "~followup_annotations": [
-      "step returned non-zero exit code: 1",
       "@@@STEP_TEXT@foo@@@",
       "@@@STEP_FAILURE@@@",
       "@@@SET_BUILD_PROPERTY@failure_hash@\"c2311ad770732eade3d2f48247abd147e40a70e7\"@@@"
     ]
   },
   {
-    "name": "$result",
-    "reason": "Step('fail') failed with return_code 1",
-    "recipe_result": null,
-    "status_code": 1
+    "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

@@ -14,7 +14,8 @@
     "stdout": "/path/to/tmp/",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@files@sub/project/foo.cc@@@",
-      "@@@STEP_LOG_END@files@@@"
+      "@@@STEP_LOG_END@files@@@",
+      "@@@SET_BUILD_PROPERTY@do_not_retry@true@@@"
     ]
   },
   {
@@ -25,16 +26,18 @@
     ],
     "name": "fail",
     "~followup_annotations": [
-      "step returned non-zero exit code: 1",
       "@@@STEP_TEXT@foo@@@",
       "@@@STEP_FAILURE@@@",
       "@@@SET_BUILD_PROPERTY@failure_hash@\"c2311ad770732eade3d2f48247abd147e40a70e7\"@@@"
     ]
   },
   {
-    "name": "$result",
-    "reason": "Step('fail') failed with return_code 1",
-    "recipe_result": null,
-    "status_code": 1
+    "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

@@ -3,6 +3,8 @@
 # found in the LICENSE file.
 
 DEPS = [
+  'gerrit',
+  'recipe_engine/buildbucket',
   'recipe_engine/json',
   'recipe_engine/raw_io',
   'recipe_engine/path',
@@ -28,6 +30,13 @@ def RunSteps(api):
             'Foo', api.properties['patch_text']))])
     return
 
+  if api.tryserver.gerrit_change:
+    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')
+    assert api.tryserver.gerrit_change_target_ref == expected_target_ref
+
   if api.tryserver.is_gerrit_issue:
     api.tryserver.get_footers()
   api.tryserver.get_files_affected_by_patch(
@@ -37,6 +46,7 @@ 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()
@@ -52,7 +62,7 @@ def GenTests(api):
       '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.calculate_patch_root().
+  # patch root. Normal users would use gclient.get_gerrit_patch_root().
   yield (api.test('with_git_patch') +
          api.properties(
              path_config='buildbot',
@@ -75,7 +85,22 @@ def GenTests(api):
          api.properties(test_patch_root=''))
 
   yield (api.test('with_gerrit_patch') +
-         api.properties.tryserver(gerrit_project='infra/infra'))
+         api.buildbucket.try_build(
+            'chromium',
+            'linux',
+            git_repo='https://chromium.googlesource.com/chromium/src',
+            change_number=91827,
+            patch_set=1))
+
+  yield (api.test('with_gerrit_patch_and_target_ref') +
+         api.buildbucket.try_build(
+            'chromium',
+            'linux',
+            git_repo='https://chromium.googlesource.com/chromium/src',
+            change_number=91827,
+            patch_set=1) +
+         api.properties(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) +
          api.properties(test_patch_root='sub\\project'))

data/vendor/depot_tools/recipes/recipe_modules/tryserver/test_api.py

@@ -0,0 +1,14 @@
+# Copyright 2018 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 recipe_test_api
+
+
+class TryserverTestApi(recipe_test_api.RecipeTestApi):
+
+  @recipe_test_api.mod_test_data
+  @staticmethod
+  def gerrit_change_target_ref(target_ref):
+    assert target_ref.startswith('refs/')
+    return target_ref

data/vendor/depot_tools/recipes/recipe_modules/windows_sdk/__init__.py

@@ -0,0 +1,25 @@
+# Copyright 2018 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.
+
+DEPS = [
+    'recipe_engine/cipd',
+    'recipe_engine/context',
+    'recipe_engine/json',
+    'recipe_engine/path',
+    'recipe_engine/step',
+]
+
+from recipe_engine.recipe_api import Property
+from recipe_engine.config import ConfigGroup, Single
+
+PROPERTIES = {
+  '$depot_tools/windows_sdk': Property(
+    help='Properties specifically for the infra windows_sdk module.',
+    param_name='sdk_properties',
+    kind=ConfigGroup(
+      # CIPD instance ID, tag or ref for the Windows SDK version.
+      version=Single(str),
+    ), default={'version': 'uploaded:2018-06-13'},
+  )
+}

data/vendor/depot_tools/recipes/recipe_modules/windows_sdk/api.py

@@ -0,0 +1,137 @@
+# Copyright 2018 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.
+
+"""The `windows_sdk` module provides safe functions to access a hermetic
+Microsoft Visual Studio installation.
+
+Available only to Google-run bots."""
+
+import collections
+from contextlib import contextmanager
+
+from recipe_engine import recipe_api
+
+
+class WindowsSDKApi(recipe_api.RecipeApi):
+  """API for using Windows SDK distributed via CIPD."""
+
+  SDKPaths = collections.namedtuple('SDKPaths', ['win_sdk', 'dia_sdk'])
+
+  def __init__(self, sdk_properties, *args, **kwargs):
+    super(WindowsSDKApi, self).__init__(*args, **kwargs)
+
+    self._sdk_properties = sdk_properties
+
+  @contextmanager
+  def __call__(self, path=None, version=None, enabled=True, target_arch='x64'):
+    """Setups the SDK environment when enabled.
+
+    Args:
+      path (path): Path to a directory where to install the SDK
+        (default is '[CACHE]/windows_sdk')
+      version (str): CIPD version of the SDK
+        (default is set via $infra/windows_sdk.version property)
+      enabled (bool): Whether the SDK should be used or not.
+      target_arch (str): 'x86' or 'x64'.
+
+    Yields:
+      If enabled, yields SDKPaths object with paths to well-known roots within
+      the deployed bundle:
+        * win_sdk - a Path to the root of the extracted Windows SDK.
+        * dia_sdk - a Path to the root of the extracted Debug Interface Access
+          SDK.
+
+    Raises:
+        StepFailure or InfraFailure.
+    """
+    if enabled:
+      sdk_dir = self._ensure_sdk(
+          path or self.m.path['cache'].join('windows_sdk'),
+          version or self._sdk_properties['version'])
+      try:
+        with self.m.context(**self._sdk_env(sdk_dir, target_arch)):
+          yield WindowsSDKApi.SDKPaths(
+              sdk_dir.join('win_sdk'),
+              sdk_dir.join('DIA SDK'))
+      finally:
+        # cl.exe automatically starts background mspdbsrv.exe daemon which
+        # needs to be manually stopped so Swarming can tidy up after itself.
+        #
+        # Since mspdbsrv may not actually be running, don't fail if we can't
+        # actually kill it.
+        self.m.step('taskkill mspdbsrv',
+                    ['taskkill.exe', '/f', '/t', '/im', 'mspdbsrv.exe'],
+                    ok_ret='any')
+    else:
+      yield
+
+  def _ensure_sdk(self, sdk_dir, sdk_version):
+    """Ensures the Windows SDK CIPD package is installed.
+
+    Returns the directory where the SDK package has been installed.
+
+    Args:
+      path (path): Path to a directory.
+      version (str): CIPD instance ID, tag or ref.
+    """
+    with self.m.context(infra_steps=True):
+      pkgs = self.m.cipd.EnsureFile()
+      pkgs.add_package('chrome_internal/third_party/sdk/windows', sdk_version)
+      self.m.cipd.ensure(sdk_dir, pkgs)
+      return sdk_dir
+
+  def _sdk_env(self, sdk_dir, target_arch):
+    """Constructs the environment for the SDK.
+
+    Returns environment and environment prefixes.
+
+    Args:
+      sdk_dir (path): Path to a directory containing the SDK.
+      target_arch (str): 'x86' or 'x64'
+    """
+    env = {}
+    env_prefixes = {}
+
+    # Load .../win_sdk/bin/SetEnv.${arch}.json to extract the required
+    # environment. It contains a dict that looks like this:
+    # {
+    #   "env": {
+    #     "VAR": [["..", "..", "x"], ["..", "..", "y"]],
+    #     ...
+    #   }
+    # }
+    # All these environment variables need to be added to the environment
+    # for the compiler and linker to work.
+    assert target_arch in ('x86', 'x64')
+    filename = 'SetEnv.%s.json' % target_arch
+    step_result = self.m.json.read(
+        'read %s' % filename, sdk_dir.join('win_sdk', 'bin', filename),
+        step_test_data=lambda: self.m.json.test_api.output({
+            'env': {
+                'PATH': [['..', '..', 'win_sdk', 'bin', 'x64']],
+                'VSINSTALLDIR': [['..', '..\\']],
+            },
+        }))
+    data = step_result.json.output.get('env')
+    for key in data:
+      # recipes' Path() does not like .., ., \, or /, so this is cumbersome.
+      # What we want to do is:
+      #   [sdk_bin_dir.join(*e) for e in env[k]]
+      # Instead do that badly, and rely (but verify) on the fact that the paths
+      # are all specified relative to the root, but specified relative to
+      # win_sdk/bin (i.e. everything starts with "../../".)
+      results = []
+      for value in data[key]:
+        assert value[0] == '..' and (value[1] == '..' or value[1] == '..\\')
+        results.append('%s' % sdk_dir.join(*value[2:]))
+
+      # PATH is special-cased because we don't want to overwrite other things
+      # like C:\Windows\System32. Others are replacements because prepending
+      # doesn't necessarily makes sense, like VSINSTALLDIR.
+      if key.lower() == 'path':
+        env_prefixes[key] = results
+      else:
+        env[key] = ';'.join(results)
+
+    return {'env': env, 'env_prefixes': env_prefixes}