Format TODO URLs as TODO(crbug.com/openscreen/000)
Put crbug URLs in TODOs to make it easier to open the bug pages.
Change-Id: Ibc8fce98a795d7e8c714627bf1ecd970a3865563
Reviewed-on: https://chromium-review.googlesource.com/c/openscreen/+/1872176
Reviewed-by: Max Yakimakha <yakimakha@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
diff --git a/platform/api/internal/trace_logging_internal_unittest.cc b/platform/api/internal/trace_logging_internal_unittest.cc
index b8e6507..9cbce70 100644
--- a/platform/api/internal/trace_logging_internal_unittest.cc
+++ b/platform/api/internal/trace_logging_internal_unittest.cc
@@ -12,7 +12,8 @@
#include "platform/api/trace_logging.h"
#include "platform/test/trace_logging_helpers.h"
-// TODO(issue/52): Remove duplicate code from trace logging+internal unit tests
+// TODO(crbug.com/openscreen/52): Remove duplicate code from trace
+// logging+internal unit tests
namespace openscreen {
namespace platform {
namespace internal {
diff --git a/platform/api/tls_connection.cc b/platform/api/tls_connection.cc
index ee0258c..e233a00 100644
--- a/platform/api/tls_connection.cc
+++ b/platform/api/tls_connection.cc
@@ -15,7 +15,7 @@
}
task_runner_->PostTask([this]() {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnWriteBlocked(this);
});
}
@@ -26,7 +26,7 @@
}
task_runner_->PostTask([this]() {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnWriteUnblocked(this);
});
}
@@ -37,7 +37,7 @@
}
task_runner_->PostTask([e = std::move(error), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnError(this, std::move(e));
});
}
@@ -48,7 +48,7 @@
}
task_runner_->PostTask([b = std::move(block), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnRead(this, std::move(b));
});
}
diff --git a/platform/api/tls_connection_factory.cc b/platform/api/tls_connection_factory.cc
index 8a74b1e..ee1f9ab 100644
--- a/platform/api/tls_connection_factory.cc
+++ b/platform/api/tls_connection_factory.cc
@@ -12,7 +12,7 @@
std::unique_ptr<TlsConnection> connection) {
task_runner_->PostTask(
[peer_cert, c = std::move(connection), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnAccepted(this, peer_cert, std::move(c));
});
}
@@ -22,7 +22,7 @@
std::unique_ptr<TlsConnection> connection) {
task_runner_->PostTask(
[peer_cert, c = std::move(connection), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnConnected(this, peer_cert, std::move(c));
});
}
@@ -30,14 +30,14 @@
void TlsConnectionFactory::OnConnectionFailed(
const IPEndpoint& remote_address) {
task_runner_->PostTask([remote_address, this]() {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnConnectionFailed(this, remote_address);
});
}
void TlsConnectionFactory::OnError(Error error) {
task_runner_->PostTask([e = std::move(error), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnError(this, std::move(e));
});
}
diff --git a/platform/api/trace_logging_unittest.cc b/platform/api/trace_logging_unittest.cc
index 05121f5..6a6e365 100644
--- a/platform/api/trace_logging_unittest.cc
+++ b/platform/api/trace_logging_unittest.cc
@@ -14,7 +14,8 @@
#include "platform/api/trace_logging.h"
#include "platform/test/trace_logging_helpers.h"
-// TODO(issue/52): Remove duplicate code from trace logging+internal unit tests
+// TODO(crbug.com/openscreen/52): Remove duplicate code from trace
+// logging+internal unit tests
namespace openscreen {
namespace platform {
namespace {
diff --git a/platform/api/udp_socket.cc b/platform/api/udp_socket.cc
index 263d300..3c75b18 100644
--- a/platform/api/udp_socket.cc
+++ b/platform/api/udp_socket.cc
@@ -38,7 +38,7 @@
}
task_runner_->PostTask([e = std::move(error), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnError(this, std::move(e));
});
}
@@ -49,7 +49,7 @@
}
task_runner_->PostTask([e = std::move(error), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnSendError(this, std::move(e));
});
}
@@ -60,7 +60,7 @@
}
task_runner_->PostTask([data = std::move(read_data), this]() mutable {
- // TODO(issues/71): |this| may be invalid at this point.
+ // TODO(crbug.com/openscreen/71): |this| may be invalid at this point.
this->client_->OnRead(this, std::move(data));
});
}