LSC: Replace deprecated thread annotations macros. Abseil thread annotation macros are now prefixed by ABSL_. There is no semantic change; this is just a rename. The edits were generated by devtools/cymbal/tools/run_clang_mr_refactor.sh LSC documentation: go/absl-prefixed-thread-annotations-lsc LSC approval http://b/135729553 Tested: TGP Tested: TAP train for global presubmit queue http://test/OCL:265857045:BASE:265860626:1567005016843:25ab0419 PiperOrigin-RevId: 265940959 Change-Id: Ib1c6cdae4330696aca80b221f5a0df143dc3d15a
diff --git a/quic/core/crypto/crypto_secret_boxer.h b/quic/core/crypto/crypto_secret_boxer.h index 395d389..d3ae035 100644 --- a/quic/core/crypto/crypto_secret_boxer.h +++ b/quic/core/crypto/crypto_secret_boxer.h
@@ -59,7 +59,7 @@ // state_ is an opaque pointer to whatever additional state the concrete // implementation of CryptoSecretBoxer requires. - std::unique_ptr<State> state_ GUARDED_BY(lock_); + std::unique_ptr<State> state_ ABSL_GUARDED_BY(lock_); }; } // namespace quic
diff --git a/quic/core/crypto/quic_crypto_server_config.h b/quic/core/crypto/quic_crypto_server_config.h index d8d9cab..f9ace68 100644 --- a/quic/core/crypto/quic_crypto_server_config.h +++ b/quic/core/crypto/quic_crypto_server_config.h
@@ -508,7 +508,7 @@ // Get a ref to the config with a given server config id. QuicReferenceCountedPointer<Config> GetConfigWithScid( QuicStringPiece requested_scid) const - SHARED_LOCKS_REQUIRED(configs_lock_); + ABSL_SHARED_LOCKS_REQUIRED(configs_lock_); // A snapshot of the configs associated with an in-progress handshake. struct Configs { @@ -537,7 +537,7 @@ // SelectNewPrimaryConfig reevaluates the primary config based on the // "primary_time" deadlines contained in each. void SelectNewPrimaryConfig(QuicWallTime now) const - EXCLUSIVE_LOCKS_REQUIRED(configs_lock_); + ABSL_EXCLUSIVE_LOCKS_REQUIRED(configs_lock_); // EvaluateClientHello checks |client_hello_state->client_hello| for gross // errors and determines whether it is fresh (i.e. not a replay). The results @@ -850,7 +850,7 @@ // Returns true if the next config promotion should happen now. bool IsNextConfigReady(QuicWallTime now) const - SHARED_LOCKS_REQUIRED(configs_lock_); + ABSL_SHARED_LOCKS_REQUIRED(configs_lock_); // replay_protection_ controls whether the server enforces that handshakes // aren't replays. @@ -869,12 +869,12 @@ // configs_ contains all active server configs. It's expected that there are // about half-a-dozen configs active at any one time. - ConfigMap configs_ GUARDED_BY(configs_lock_); + ConfigMap configs_ ABSL_GUARDED_BY(configs_lock_); // primary_config_ points to a Config (which is also in |configs_|) which is // the primary config - i.e. the one that we'll give out to new clients. mutable QuicReferenceCountedPointer<Config> primary_config_ - GUARDED_BY(configs_lock_); + ABSL_GUARDED_BY(configs_lock_); // fallback_config_ points to a Config (which is also in |configs_|) which is // the fallback config, which will be used if the other configs are unuseable @@ -882,15 +882,16 @@ // // TODO(b/112548056): This is currently always nullptr. QuicReferenceCountedPointer<Config> fallback_config_ - GUARDED_BY(configs_lock_); + ABSL_GUARDED_BY(configs_lock_); // next_config_promotion_time_ contains the nearest, future time when an // active config will be promoted to primary. - mutable QuicWallTime next_config_promotion_time_ GUARDED_BY(configs_lock_); + mutable QuicWallTime next_config_promotion_time_ + ABSL_GUARDED_BY(configs_lock_); // Callback to invoke when the primary config changes. std::unique_ptr<PrimaryConfigChangedCallback> primary_config_changed_cb_ - GUARDED_BY(configs_lock_); + ABSL_GUARDED_BY(configs_lock_); // Used to protect the source-address tokens that are given to clients. CryptoSecretBoxer source_address_token_boxer_;
diff --git a/quic/platform/api/quic_mutex.h b/quic/platform/api/quic_mutex.h index 162863a..86e8f67 100644 --- a/quic/platform/api/quic_mutex.h +++ b/quic/platform/api/quic_mutex.h
@@ -10,28 +10,28 @@ namespace quic { // A class representing a non-reentrant mutex in QUIC. -class LOCKABLE QUIC_EXPORT_PRIVATE QuicMutex { +class ABSL_LOCKABLE QUIC_EXPORT_PRIVATE QuicMutex { public: QuicMutex() = default; QuicMutex(const QuicMutex&) = delete; QuicMutex& operator=(const QuicMutex&) = delete; // Block until this Mutex is free, then acquire it exclusively. - void WriterLock() EXCLUSIVE_LOCK_FUNCTION(); + void WriterLock() ABSL_EXCLUSIVE_LOCK_FUNCTION(); // Release this Mutex. Caller must hold it exclusively. - void WriterUnlock() UNLOCK_FUNCTION(); + void WriterUnlock() ABSL_UNLOCK_FUNCTION(); // Block until this Mutex is free or shared, then acquire a share of it. - void ReaderLock() SHARED_LOCK_FUNCTION(); + void ReaderLock() ABSL_SHARED_LOCK_FUNCTION(); // Release this Mutex. Caller could hold it in shared mode. - void ReaderUnlock() UNLOCK_FUNCTION(); + void ReaderUnlock() ABSL_UNLOCK_FUNCTION(); // Returns immediately if current thread holds the Mutex in at least shared // mode. Otherwise, may report an error (typically by crashing with a // diagnostic), or may return immediately. - void AssertReaderHeld() const ASSERT_SHARED_LOCK(); + void AssertReaderHeld() const ABSL_ASSERT_SHARED_LOCK(); private: QuicLockImpl impl_; @@ -39,13 +39,13 @@ // A helper class that acquires the given QuicMutex shared lock while the // QuicReaderMutexLock is in scope. -class SCOPED_LOCKABLE QUIC_EXPORT_PRIVATE QuicReaderMutexLock { +class ABSL_SCOPED_LOCKABLE QUIC_EXPORT_PRIVATE QuicReaderMutexLock { public: - explicit QuicReaderMutexLock(QuicMutex* lock) SHARED_LOCK_FUNCTION(lock); + explicit QuicReaderMutexLock(QuicMutex* lock) ABSL_SHARED_LOCK_FUNCTION(lock); QuicReaderMutexLock(const QuicReaderMutexLock&) = delete; QuicReaderMutexLock& operator=(const QuicReaderMutexLock&) = delete; - ~QuicReaderMutexLock() UNLOCK_FUNCTION(); + ~QuicReaderMutexLock() ABSL_UNLOCK_FUNCTION(); private: QuicMutex* const lock_; @@ -53,13 +53,14 @@ // A helper class that acquires the given QuicMutex exclusive lock while the // QuicWriterMutexLock is in scope. -class SCOPED_LOCKABLE QUIC_EXPORT_PRIVATE QuicWriterMutexLock { +class ABSL_SCOPED_LOCKABLE QUIC_EXPORT_PRIVATE QuicWriterMutexLock { public: - explicit QuicWriterMutexLock(QuicMutex* lock) EXCLUSIVE_LOCK_FUNCTION(lock); + explicit QuicWriterMutexLock(QuicMutex* lock) + ABSL_EXCLUSIVE_LOCK_FUNCTION(lock); QuicWriterMutexLock(const QuicWriterMutexLock&) = delete; QuicWriterMutexLock& operator=(const QuicWriterMutexLock&) = delete; - ~QuicWriterMutexLock() UNLOCK_FUNCTION(); + ~QuicWriterMutexLock() ABSL_UNLOCK_FUNCTION(); private: QuicMutex* const lock_;
diff --git a/quic/test_tools/packet_dropping_test_writer.h b/quic/test_tools/packet_dropping_test_writer.h index 92c8956..bbccd78 100644 --- a/quic/test_tools/packet_dropping_test_writer.h +++ b/quic/test_tools/packet_dropping_test_writer.h
@@ -161,14 +161,14 @@ uint64_t num_calls_to_write_; QuicMutex config_mutex_; - int32_t fake_packet_loss_percentage_ GUARDED_BY(config_mutex_); - int32_t fake_drop_first_n_packets_ GUARDED_BY(config_mutex_); - int32_t fake_blocked_socket_percentage_ GUARDED_BY(config_mutex_); - int32_t fake_packet_reorder_percentage_ GUARDED_BY(config_mutex_); - QuicTime::Delta fake_packet_delay_ GUARDED_BY(config_mutex_); - QuicBandwidth fake_bandwidth_ GUARDED_BY(config_mutex_); - QuicByteCount buffer_size_ GUARDED_BY(config_mutex_); - int32_t num_consecutive_packet_lost_ GUARDED_BY(config_mutex_); + int32_t fake_packet_loss_percentage_ ABSL_GUARDED_BY(config_mutex_); + int32_t fake_drop_first_n_packets_ ABSL_GUARDED_BY(config_mutex_); + int32_t fake_blocked_socket_percentage_ ABSL_GUARDED_BY(config_mutex_); + int32_t fake_packet_reorder_percentage_ ABSL_GUARDED_BY(config_mutex_); + QuicTime::Delta fake_packet_delay_ ABSL_GUARDED_BY(config_mutex_); + QuicBandwidth fake_bandwidth_ ABSL_GUARDED_BY(config_mutex_); + QuicByteCount buffer_size_ ABSL_GUARDED_BY(config_mutex_); + int32_t num_consecutive_packet_lost_ ABSL_GUARDED_BY(config_mutex_); }; } // namespace test
diff --git a/quic/test_tools/quic_crypto_server_config_peer.h b/quic/test_tools/quic_crypto_server_config_peer.h index 3f5a984..6eed475 100644 --- a/quic/test_tools/quic_crypto_server_config_peer.h +++ b/quic/test_tools/quic_crypto_server_config_peer.h
@@ -73,7 +73,7 @@ // ConfigsDebug returns a std::string that contains debugging information // about the set of Configs loaded in |server_config_| and their status. std::string ConfigsDebug() - SHARED_LOCKS_REQUIRED(server_config_->configs_lock_); + ABSL_SHARED_LOCKS_REQUIRED(server_config_->configs_lock_); void SelectNewPrimaryConfig(int seconds);
diff --git a/quic/test_tools/server_thread.h b/quic/test_tools/server_thread.h index ffbce7f..a55ecb3 100644 --- a/quic/test_tools/server_thread.h +++ b/quic/test_tools/server_thread.h
@@ -73,13 +73,13 @@ std::unique_ptr<QuicServer> server_; QuicSocketAddress address_; mutable QuicMutex port_lock_; - int port_ GUARDED_BY(port_lock_); + int port_ ABSL_GUARDED_BY(port_lock_); bool initialized_; QuicMutex scheduled_actions_lock_; QuicDeque<std::function<void()>> scheduled_actions_ - GUARDED_BY(scheduled_actions_lock_); + ABSL_GUARDED_BY(scheduled_actions_lock_); }; } // namespace test
diff --git a/quic/tools/quic_memory_cache_backend.h b/quic/tools/quic_memory_cache_backend.h index ff76c9a..7249bc0 100644 --- a/quic/tools/quic_memory_cache_backend.h +++ b/quic/tools/quic_memory_cache_backend.h
@@ -177,15 +177,15 @@ // Cached responses. QuicUnorderedMap<std::string, std::unique_ptr<QuicBackendResponse>> responses_ - GUARDED_BY(response_mutex_); + ABSL_GUARDED_BY(response_mutex_); // The default response for cache misses, if set. std::unique_ptr<QuicBackendResponse> default_response_ - GUARDED_BY(response_mutex_); + ABSL_GUARDED_BY(response_mutex_); // A map from request URL to associated server push responses (if any). std::multimap<std::string, QuicBackendResponse::ServerPushInfo> - server_push_resources_ GUARDED_BY(response_mutex_); + server_push_resources_ ABSL_GUARDED_BY(response_mutex_); // Protects against concurrent access from test threads setting responses, and // server threads accessing those responses.