[fxbug.dev] Migrate bug numbers
This changes fxbug.dev/ URLs from Monorail bug numbers to the new
Fuchsia Issue Tracker numbers.
The migration to the new issue tracker was announced here:
https://groups.google.com/a/fuchsia.dev/g/announce/c/GOYfJozEqmk/m/qsGsaJ7UAAAJ
Bug: 298074672
Change-Id: If46152cf199868bd3692ebc7a12df614bec4c3be
diff --git a/guest/GoldfishAddressSpace/include/goldfish_address_space_fuchsia.impl b/guest/GoldfishAddressSpace/include/goldfish_address_space_fuchsia.impl
index 57e394b..c8a7ae9 100644
--- a/guest/GoldfishAddressSpace/include/goldfish_address_space_fuchsia.impl
+++ b/guest/GoldfishAddressSpace/include/goldfish_address_space_fuchsia.impl
@@ -133,7 +133,7 @@
return false;
}
m_phys_addr = result->paddr;
- // TODO(fxbug.dev/124700): Instead of storing raw handles we should
+ // TODO(fxbug.dev/42075554): Instead of storing raw handles we should
// consider using RAII zx::object instead.
m_vmo = result->vmo.release();
@@ -363,7 +363,7 @@
*offset = 0;
- // TODO(fxbug.dev/124700): Instead of storing raw handles we should
+ // TODO(fxbug.dev/42075554): Instead of storing raw handles we should
// consider using RAII zx::object instead.
VmoStore::Info info = {
vmo.release(),
@@ -408,7 +408,7 @@
}
vmo = std::move(result->vmo);
- // TODO(fxbug.dev/124700): Instead of storing raw handles we should
+ // TODO(fxbug.dev/42075554): Instead of storing raw handles we should
// consider using RAII zx::object instead.
VmoStore::Info info = {
vmo.release(),
diff --git a/guest/android-emu/aemu/base/fit/Nullable.h b/guest/android-emu/aemu/base/fit/Nullable.h
index 02b3337..06e4527 100644
--- a/guest/android-emu/aemu/base/fit/Nullable.h
+++ b/guest/android-emu/aemu/base/fit/Nullable.h
@@ -72,7 +72,7 @@
// - sizeof(fit::nullable<int>) == sizeof(struct { bool; int; })
// - sizeof(std::optional<int>) == sizeof(struct { bool; int; })
//
-// TODO(fxbug.dev/4681): fit::nullable does not precisely mirror
+// TODO(fxbug.dev/42123486): fit::nullable does not precisely mirror
// std::optional. This should be corrected to avoid surprises when switching
// between the types.
template <typename T,
diff --git a/guest/magma/magma.cpp b/guest/magma/magma.cpp
index ca4c12e..6f256a6 100644
--- a/guest/magma/magma.cpp
+++ b/guest/magma/magma.cpp
@@ -154,7 +154,7 @@
}
auto& info = it->second;
- // TODO(fxbug.dev/122604): Evaluate deferred guest resource creation.
+ // TODO(fxbug.dev/42073573): Evaluate deferred guest resource creation.
auto blob = VirtGpuDevice::getInstance(VirtGpuCapset::kCapsetGfxStreamMagma)
->createBlob({.size = info.size,
.flags = kBlobFlagMappable | kBlobFlagShareable,
diff --git a/guest/magma/magma_enc/magma_enc.cpp b/guest/magma/magma_enc/magma_enc.cpp
index f181eee..7f865c6 100644
--- a/guest/magma/magma_enc/magma_enc.cpp
+++ b/guest/magma/magma_enc/magma_enc.cpp
@@ -12,7 +12,7 @@
#include <stdio.h>
-// TODO(fxbug.dev/122605): Why is modification needed for these two includes?
+// TODO(fxbug.dev/42073574): Why is modification needed for these two includes?
#include "aemu/base/Tracing.h"
#include "cutils/log.h"
diff --git a/guest/vulkan_enc/ResourceTracker.cpp b/guest/vulkan_enc/ResourceTracker.cpp
index eef58e6..a2fd306 100644
--- a/guest/vulkan_enc/ResourceTracker.cpp
+++ b/guest/vulkan_enc/ResourceTracker.cpp
@@ -489,7 +489,7 @@
}
}
-// TODO(fxbug.dev/90856): This is currently only used for allocating
+// TODO(fxbug.dev/42172354): This is currently only used for allocating
// memory for dedicated external images. It should be migrated to use
// SetBufferCollectionImageConstraintsFUCHSIA.
VkResult ResourceTracker::setBufferCollectionConstraintsFUCHSIA(
@@ -2299,7 +2299,7 @@
// Importing read-only host memory into the Vulkan driver should not
// work, but it is not an error to try to do so. Returning a
// VkMemoryZirconHandlePropertiesFUCHSIA with no available
- // memoryType bits should be enough for clients. See fxbug.dev/24225
+ // memoryType bits should be enough for clients. See fxbug.dev/42098398
// for other issues this this flow.
ALOGW("GetBufferHandleInfo failed: %d", result.value().error_value());
pProperties->memoryTypeBits = 0;
@@ -2531,7 +2531,7 @@
ALOGW(
"%s: Non-zero flags (%08x) in image format "
"constraints; this is currently not supported, see "
- "fxbug.dev/68833.",
+ "fxbug.dev/42147900.",
__func__, formatConstraints->flags);
}
}
@@ -3560,7 +3560,7 @@
fidl::WireSyncClient<fuchsia_sysmem::BufferCollection> collection(
std::move(collection_ends->client));
if (hasDedicatedImage) {
- // TODO(fxbug.dev/90856): Use setBufferCollectionImageConstraintsFUCHSIA.
+ // TODO(fxbug.dev/42172354): Use setBufferCollectionImageConstraintsFUCHSIA.
VkResult res = setBufferCollectionConstraintsFUCHSIA(enc, device, &collection,
pImageCreateInfo);
if (res == VK_ERROR_FORMAT_NOT_SUPPORTED) {
@@ -5781,7 +5781,7 @@
post_wait_events.push_back({semInfo.eventHandle, semInfo.eventKoid});
#ifndef FUCHSIA_NO_TRACE
if (semInfo.eventKoid != ZX_KOID_INVALID) {
- // TODO(fxbug.dev/66098): Remove the "semaphore"
+ // TODO(fxbug.dev/42144867): Remove the "semaphore"
// FLOW_END events once it is removed from clients
// (for example, gfx Engine).
TRACE_FLOW_END("gfx", "semaphore", semInfo.eventKoid);
diff --git a/third-party/fuchsia/README b/third-party/fuchsia/README
index e315413..e012b39 100644
--- a/third-party/fuchsia/README
+++ b/third-party/fuchsia/README
@@ -1 +1 @@
-These files come from fuchsia.googlesource.com. Once Magma is in the Fuchsia SDK (https://fxbug.dev/116007, https://fxbug.dev/111471), these files should be removed and the Fuchsia SDK added as a proper dependency (https://fxbug.dev/122678).
+These files come from fuchsia.googlesource.com. Once Magma is in the Fuchsia SDK (https://fxbug.dev/42067270, https://fxbug.dev/42062806), these files should be removed and the Fuchsia SDK added as a proper dependency (https://fxbug.dev/42073654).
