commit 9570c8ceeaa562cc08225e9ac0cb743c36bfe073
author Alison Gale <agale@chromium.org> Thu Apr 25 20:48:29 2024
committer Copybara-Service <copybara-worker@google.com> Thu Apr 25 20:54:48 2024
tree 784af9078dae4573190cc025417ff29db32a2660
parent 949930758ec6653735a9a4def4e1a460a9e37cca

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: I70c4664df7ce08868073fc49d28692fe742fd5eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5487422
Commit-Queue: Peter Boström <pbos@chromium.org>
Owners-Override: Alison Gale <agale@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Auto-Submit: Alison Gale <agale@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1292683}
NOKEYCHECK=True
GitOrigin-RevId: 9c9683f0bd1449306a792b278a6c4c3f822bf730

scripts/analyze_includes.py

diff --git a/scripts/analyze_includes.py b/scripts/analyze_includes.py
index bb661cf..f9121d0 100755
--- a/scripts/analyze_includes.py
+++ b/scripts/analyze_includes.py
@@ -92,7 +92,7 @@
 
       if depth > prev_depth:
         if sys.platform != 'win32':
-          # TODO(crbug.com/1223741): Always assert.
+          # TODO(crbug.com/40187759): Always assert.
           assert depth == prev_depth + 1
         elif depth > prev_depth + 1:
           # Until the bug is fixed, skip these includes.

scripts/build.py

diff --git a/scripts/build.py b/scripts/build.py
index 9817654..8af5957 100755
--- a/scripts/build.py
+++ b/scripts/build.py
@@ -1201,7 +1201,7 @@
     runtimes_triples_args['i386-unknown-linux-gnu'] = {
         "args": [
             'CMAKE_SYSROOT=%s' % sysroot_i386,
-            # TODO(https://crbug.com/1374690): pass proper flags to i386 tests so they compile correctly
+            # TODO(crbug.com/40242553): pass proper flags to i386 tests so they compile correctly
             'LLVM_INCLUDE_TESTS=OFF',
         ],
         "profile":

scripts/package.py

diff --git a/scripts/package.py b/scripts/package.py
index 1f83a73..3b80e5f 100755
--- a/scripts/package.py
+++ b/scripts/package.py
@@ -105,7 +105,7 @@
         tar_xz.add(os.path.join(directory_path, f),
                    arcname=f,
                    filter=PrintTarProgress)
-        # TODO(crbug.com/1261812) Stop making gzip'ed archives once the
+        # TODO(crbug.com/40202359) Stop making gzip'ed archives once the
         # goma/reclient push processes are updated to consume the .xz files
         # instead.
         tar_gz.add(os.path.join(directory_path, f), arcname=f)

scripts/remote_ld.py

diff --git a/scripts/remote_ld.py b/scripts/remote_ld.py
index 3ed2111..c7d4ce7 100755
--- a/scripts/remote_ld.py
+++ b/scripts/remote_ld.py
@@ -35,7 +35,7 @@
   }
 
   def analyze_args(self, args, *posargs, **kwargs):
-    # TODO(crbug.com/1040196): Builds are unreliable when all targets use
+    # TODO(crbug.com/40113922): Builds are unreliable when all targets use
     # distributed ThinLTO, so we only enable it for some targets.
     # For other targets, we fall back to local ThinLTO. We must use ThinLTO
     # because we build with -fsplit-lto-unit, which requires code generation