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: Iee14d10d544e9f0ec046117cc4ec8a55c427adc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5469947
Reviewed-by: Darryl James <[email protected]>
Owners-Override: Alison Gale <[email protected]>
Commit-Queue: Alison Gale <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1290838}
diff --git a/content/public/browser/authenticator_request_client_delegate.cc b/content/public/browser/authenticator_request_client_delegate.cc
index 26b3b74..a418f9f 100644
--- a/content/public/browser/authenticator_request_client_delegate.cc
+++ b/content/public/browser/authenticator_request_client_delegate.cc
@@ -170,7 +170,7 @@
callback) {
// Automatically choose the first account to allow resident keys for virtual
// authenticators without a browser implementation, e.g. on content shell.
- // TODO(crbug.com/991666): Provide a way to determine which account gets
+ // TODO(crbug.com/40639383): Provide a way to determine which account gets
// picked.
DCHECK(virtual_environment_);
std::move(callback).Run(std::move(responses.at(0)));
diff --git a/content/public/browser/browsing_data_remover.h b/content/public/browser/browsing_data_remover.h
index 78e7fae82..6ff14b9 100644
--- a/content/public/browser/browsing_data_remover.h
+++ b/content/public/browser/browsing_data_remover.h
@@ -59,8 +59,8 @@
//
////////////////////////////////////////////////////////////////////////////////
//
-// TODO(crbug.com/668114): BrowsingDataRemover does not currently support plugin
-// data deletion. Use PluginDataRemover instead.
+// TODO(crbug.com/40495069): BrowsingDataRemover does not currently support
+// plugin data deletion. Use PluginDataRemover instead.
class BrowsingDataRemover {
public:
// Mask used for Remove.
diff --git a/content/public/browser/child_process_host.h b/content/public/browser/child_process_host.h
index 468e4e62..303199a 100644
--- a/content/public/browser/child_process_host.h
+++ b/content/public/browser/child_process_host.h
@@ -121,7 +121,7 @@
// Starts a child process with the macOS entitlement that allows unsigned
// executable memory.
- // TODO(https://crbug.com/985816): Change this to use MAP_JIT and the
+ // TODO(crbug.com/40636855): Change this to use MAP_JIT and the
// allow-jit entitlement instead.
CHILD_GPU,
diff --git a/content/public/browser/content_browser_client.cc b/content/public/browser/content_browser_client.cc
index f9c594c..0ea2716 100644
--- a/content/public/browser/content_browser_client.cc
+++ b/content/public/browser/content_browser_client.cc
@@ -966,7 +966,7 @@
}
bool ContentBrowserClient::ShouldEnableAudioProcessHighPriority() {
- // TODO(crbug.com/1374069): Delete this method when the
+ // TODO(crbug.com/40242320): Delete this method when the
// kAudioProcessHighPriorityEnabled enterprise policy is deprecated.
return false;
}
diff --git a/content/public/browser/document_user_data.h b/content/public/browser/document_user_data.h
index 9c4bead..55d7926d 100644
--- a/content/public/browser/document_user_data.h
+++ b/content/public/browser/document_user_data.h
@@ -134,7 +134,7 @@
RenderFrameHost& render_frame_host() const { return *render_frame_host_; }
protected:
- // TODO(https://crbug.com/1252044): Take a reference instead of a pointer
+ // TODO(crbug.com/40198594): Take a reference instead of a pointer
// (here + transitively/as-far-as-reasonably-possible in callers).
explicit DocumentUserData(RenderFrameHost* rfh) : render_frame_host_(rfh) {
CHECK(rfh);
diff --git a/content/public/browser/frame_type.h b/content/public/browser/frame_type.h
index b7a743f5..8562838 100644
--- a/content/public/browser/frame_type.h
+++ b/content/public/browser/frame_type.h
@@ -39,7 +39,7 @@
// This differs from RenderFrameHost::IsFencedFrameRoot(), which covers
// both MPArch and ShadowDOM-based FencedFrames.
kFencedFrameRoot,
- // TODO(crbug.com/1267506): Consider if we need to have the types for portals
+ // TODO(crbug.com/40204472): Consider if we need to have the types for portals
// or guestviews.
};
diff --git a/content/public/browser/host_zoom_map.h b/content/public/browser/host_zoom_map.h
index c57932f..abc331d2 100644
--- a/content/public/browser/host_zoom_map.h
+++ b/content/public/browser/host_zoom_map.h
@@ -192,7 +192,8 @@
virtual void SetDefaultZoomLevelPrefCallback(
DefaultZoomChangedCallback callback) = 0;
- // TODO(crbug.com/1424904): Make an Android-specific impl of host_zoom_map, or
+ // TODO(crbug.com/40898422): Make an Android-specific impl of host_zoom_map,
+ // or
// combine method with GetZoomLevelForHostAndScheme.
virtual double GetZoomLevelForHostAndSchemeAndroid(
const std::string& scheme,
diff --git a/content/public/browser/media_session.h b/content/public/browser/media_session.h
index fca03cc5..ebba34f 100644
--- a/content/public/browser/media_session.h
+++ b/content/public/browser/media_session.h
@@ -43,7 +43,7 @@
// Media item IDs have a shared namespace including both UnguessableTokens and
// strings.
- // TODO(https://crbug.com/1260385): Use UnguessableToken only and remove this
+ // TODO(crbug.com/40798185): Use UnguessableToken only and remove this
// API.
CONTENT_EXPORT static WebContents* GetWebContentsFromRequestId(
const std::string& request_id);
diff --git a/content/public/browser/notification_database_data.h b/content/public/browser/notification_database_data.h
index e83ee791..3aa20e0 100644
--- a/content/public/browser/notification_database_data.h
+++ b/content/public/browser/notification_database_data.h
@@ -22,7 +22,7 @@
// database. Beyond the notification's own data, its id and attribution need
// to be available for users of the database as well.
// Note: There are extra properties being stored for UKM logging purposes.
-// TODO(https://crbug.com/842622): Add the UKM that will use these properties.
+// TODO(crbug.com/40576162): Add the UKM that will use these properties.
struct CONTENT_EXPORT NotificationDatabaseData {
NotificationDatabaseData();
NotificationDatabaseData(const NotificationDatabaseData& other);
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 8403cac5..22d5ca8 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -451,8 +451,8 @@
// committing a navigation. After the first navigation commits this
// will return the token for the last committed document.
//
- // TODO(crbug/1098283): Remove the nullopt scenario by creating the token in
- // CreateChildFrame() or similar.
+ // TODO(crbug.com/40136951): Remove the nullopt scenario by creating the token
+ // in CreateChildFrame() or similar.
virtual std::optional<base::UnguessableToken> GetEmbeddingToken() = 0;
// Returns the assigned name of the frame, the name of the iframe tag
diff --git a/content/public/browser/shared_worker_instance.cc b/content/public/browser/shared_worker_instance.cc
index cc26e267..4cca061 100644
--- a/content/public/browser/shared_worker_instance.cc
+++ b/content/public/browser/shared_worker_instance.cc
@@ -58,7 +58,7 @@
return false;
}
- // TODO(https://crbug.com/794098): file:// URLs should be treated as opaque
+ // TODO(crbug.com/40554285): file:// URLs should be treated as opaque
// origins, but not in url::Origin. Therefore, we manually check it here.
if (url.SchemeIsFile() || storage_key.origin().scheme() == url::kFileScheme)
return false;
diff --git a/content/public/browser/storage_partition_config.h b/content/public/browser/storage_partition_config.h
index 08059a4..f7d4e463 100644
--- a/content/public/browser/storage_partition_config.h
+++ b/content/public/browser/storage_partition_config.h
@@ -67,7 +67,7 @@
if (fallback != FallbackMode::kNone) {
DCHECK(!is_default());
DCHECK(!partition_domain_.empty());
- // TODO(https://crbug.com/1279537): Ideally we shouldn't have storage
+ // TODO(crbug.com/40208586): Ideally we shouldn't have storage
// partition configs that differ only in their fallback mode, but
// unfortunately that isn't true. When that is fixed this can be made more
// robust by disallowing fallback from storage partitions with an empty
diff --git a/content/public/browser/url_data_source.h b/content/public/browser/url_data_source.h
index 2f07ec6..2dbf4e9 100644
--- a/content/public/browser/url_data_source.h
+++ b/content/public/browser/url_data_source.h
@@ -98,7 +98,7 @@
network::mojom::CSPDirectiveName directive);
// By default, neither of these headers are set. Override to change this.
- // TODO(https://crbug.com/1189194): Consider setting COOP:same-origin and
+ // TODO(crbug.com/40755309): Consider setting COOP:same-origin and
// COEP:require-corp as the default instead.
virtual std::string GetCrossOriginOpenerPolicy();
virtual std::string GetCrossOriginEmbedderPolicy();
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 51919f7..a987a5e 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -914,7 +914,7 @@
//
// Always returns a non-null value.
//
- // TODO(crbug.com/1498410): Consider replacing this with
+ // TODO(crbug.com/40939539): Consider replacing this with
// GuestViewBase::GetTopLevelWebContents, since that is now the only case
// where this would return a contents other than |this|.
virtual WebContents* GetResponsibleWebContents() = 0;
@@ -1513,7 +1513,7 @@
// Tag `WebContents` with its owner. Used purely for debugging purposes so it
// does not need to be exhaustive or perfectly correct.
- // TODO(crbug.com/1407197): Remove after bug is fixed.
+ // TODO(crbug.com/40062641): Remove after bug is fixed.
virtual void SetOwnerLocationForDebug(
std::optional<base::Location> owner_location) = 0;
