Fix 10 ClangTidy - Performance findings in //depot/google3/third_party/quic/.
These fixes are automatically created by various analysis tools, but their application in this CL has been manually triggered. See go/code-findings-fixes.
* passing result of std::move() as a const reference argument; no move will actually happen (4 times)
* use absl::StartsWith instead of find() == 0
* use ZeroDuration() for zero-length time intervals (5 times)
PiperOrigin-RevId: 276497039
Change-Id: Ic5461dfa0fee864263437bafa7d4a137e2b44f25
diff --git a/quic/test_tools/crypto_test_utils_test.cc b/quic/test_tools/crypto_test_utils_test.cc
index 59a3947..6565497 100644
--- a/quic/test_tools/crypto_test_utils_test.cc
+++ b/quic/test_tools/crypto_test_utils_test.cc
@@ -128,7 +128,7 @@
QuicRandom::GetInstance(), &clock, new_config_options);
primary_config.set_primary_time(clock.WallNow().ToUNIXSeconds());
std::unique_ptr<CryptoHandshakeMessage> msg =
- crypto_config.AddConfig(std::move(primary_config), clock.WallNow());
+ crypto_config.AddConfig(primary_config, clock.WallNow());
QuicStringPiece orbit;
ASSERT_TRUE(msg->GetStringPiece(kORBT, &orbit));
std::string nonce;