Rename variables of type StunMessage* from `request` to `message`.
This is just to reduce confusion since StunMessage and StunRequest
instances are frequently used together and message objects are often
configured from within request objects (which makes the name confusing).
Bug: none
Change-Id: I8bf5e774a5149239dd3023817614d411633bf583
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/258484
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Auto-Submit: Tomas Gunnarsson <tommi@webrtc.org>
Commit-Queue: Tomas Gunnarsson <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#36533}
diff --git a/p2p/base/stun_request.h b/p2p/base/stun_request.h
index e75804d..f4d6b2b 100644
--- a/p2p/base/stun_request.h
+++ b/p2p/base/stun_request.h
@@ -86,7 +86,7 @@
public:
explicit StunRequest(StunRequestManager& manager);
StunRequest(StunRequestManager& manager,
- std::unique_ptr<StunMessage> request);
+ std::unique_ptr<StunMessage> message);
~StunRequest() override;
// Causes our wrapped StunMessage to be Prepared
@@ -117,7 +117,7 @@
// Fills in a request object to be sent. Note that request's transaction ID
// will already be set and cannot be changed.
- virtual void Prepare(StunMessage* request) {}
+ virtual void Prepare(StunMessage* message) {}
// Called when the message receives a response or times out.
virtual void OnResponse(StunMessage* response) {}