Migrate TODOs referencing old crbug IDs to the new issue tracker IDs
The canonical bug format is TODO(crbug.com/<id>). TODOs of the
following forms will all be migrated to the new format:
- TODO(crbug.com/<old id>)
- TODO(https://crbug.com/<old id>)
- TODO(crbug/<old id>)
- TODO(crbug/monorail/<old id>)
- TODO(<old id>)
- TODO(issues.chromium.org/<old id>)
- TODO(https://issues.chromium.org/<old id>)
- TODO(https://issues.chromium.org/u/1/issues/<old id>)
- TODO(bugs.chromium.org/<old id>)
Bug id mapping is sourced from go/chrome-on-buganizer-prod-issues.
See go/crbug-todo-migration for details.
#crbug-todo-migration
Bug: b/321899722
Change-Id: Ib4e61022cc4ca4f0a6563a63fb5d2f582ef930c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5494425
Auto-Submit: Alison Gale <agale@chromium.org>
Owners-Override: Alison Gale <agale@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1293183}
NOKEYCHECK=True
GitOrigin-RevId: 4d9c23185ad9f590658e36154ac3e53f0f4f158b
diff --git a/README.md b/README.md
index dc47316..4a6c5d1 100644
--- a/README.md
+++ b/README.md
@@ -184,7 +184,7 @@
- Run manual tests locally (see the "Building and testing the tools locally"
section below).
- TODO(https://crbug.com/1329611): These manual steps should
+ TODO(crbug.com/40226863): These manual steps should
be made obsolete once Rust-specific tryjobs cover Crubit
tests.
@@ -233,5 +233,5 @@
Crubit tests are under `//build/rust/tests/test_rs_bindings_from_cc`. Until
Crubit is built on the bots, the tests are commented out in
`//build/rust/tests/BUILD.gn`, but they should still be built and run before
-rolling Crubit. TODO(https://crbug.com/1329611): Rephrase this paragraph
+rolling Crubit. TODO(crbug.com/40226863): Rephrase this paragraph
after Crubit is built and tested on the bots.
diff --git a/build_rust.py b/build_rust.py
index a7c48c2..060dcaf 100755
--- a/build_rust.py
+++ b/build_rust.py
@@ -147,7 +147,7 @@
OPENSSL_CIPD_MAC_AMD_VERSION = '1.1.1j.chromium.2'
OPENSSL_CIPD_MAC_ARM_PATH = 'infra/3pp/static_libs/openssl/mac-arm64'
OPENSSL_CIPD_MAC_ARM_VERSION = '1.1.1j.chromium.2'
-# TODO(crbug.com/1271215): Pull Windows OpenSSL from 3pp when it exists.
+# TODO(crbug.com/40205621): Pull Windows OpenSSL from 3pp when it exists.
if sys.platform == 'win32':
LD_PATH_FLAG = '/LIBPATH:'
@@ -706,7 +706,7 @@
else:
libxml2_dirs = None
- # TODO(crbug.com/1271215): OpenSSL is somehow already present on the Windows
+ # TODO(crbug.com/40205621): OpenSSL is somehow already present on the Windows
# builder, but we should change to using a package from 3pp when it is
# available.
if (sys.platform != 'win32' and not args.sync_for_gnrt):
diff --git a/package_rust.py b/package_rust.py
index 3de009d..fc76052 100755
--- a/package_rust.py
+++ b/package_rust.py
@@ -26,7 +26,7 @@
RUST_TOOLCHAIN_PACKAGE_NAME = f'rust-toolchain-{PACKAGE_VERSION}.tar.xz'
-# TODO(https://crbug.com/1329611): Use this function (after integrating Crubit
+# TODO(crbug.com/40226863): Use this function (after integrating Crubit
# into Chromium; this work is on hold right now - see also
# https://crbug.com/1510943#c2).
def BuildCrubit(build_mac_arm):
@@ -34,12 +34,12 @@
build_cmd = [sys.executable, os.path.join(THIS_DIR, 'build_crubit.py')]
if build_mac_arm:
build_cmd.append('--build-mac-arm')
- # TODO(https://crbug.com/1329611): Default to `fail_hard` once we
+ # TODO(crbug.com/40226863): Default to `fail_hard` once we
# actually depend on the build step (i.e. once we start packaging
# Crubit).
TeeCmd(build_cmd, log, fail_hard=False)
- # TODO(https://crbug.com/1329611): Rename this function to
+ # TODO(crbug.com/40226863): Rename this function to
# BuildAndInstallCrubit and actually install Crubit binaries into
# RUST_TOOLCHAIN_OUT_DIR/bin (once we gain confidence that Crubit continues
# to build uneventfully on the bots).
