commit 2b4ec9e667c8c5a8bac84006f1ad3dab0cc911c1
author Danil Chapovalov <danilchap@webrtc.org> Wed Mar 25 17:23:37 2020 +0100
committer Commit Bot <commit-bot@chromium.org> Thu Mar 26 14:32:45 2020 +0000
tree cbbe2965103911074a58e9018e36dea37e00aefa
parent 7ac010cea48b470e4c2d0ada3de428cf36bf34d4

in RtpExtension constructors pass strings by string_view rather than by value

To allow construct that object from an existent string_view without explicit conversion

Bug: webrtc:11428
Change-Id: I38d93573be72e307bdf7068a6300d10cf46d2d62
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/171689
Reviewed-by: Markus Handell <handellm@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30904}

api/rtp_parameters.h

diff --git a/api/rtp_parameters.h b/api/rtp_parameters.h
index 4462447..5435922 100644
--- a/api/rtp_parameters.h
+++ b/api/rtp_parameters.h
@@ -232,9 +232,9 @@
 
   // Constructors for convenience.
   RtpHeaderExtensionCapability();
-  explicit RtpHeaderExtensionCapability(std::string uri);
-  RtpHeaderExtensionCapability(std::string uri, int preferred_id);
-  RtpHeaderExtensionCapability(std::string uri,
+  explicit RtpHeaderExtensionCapability(absl::string_view uri);
+  RtpHeaderExtensionCapability(absl::string_view uri, int preferred_id);
+  RtpHeaderExtensionCapability(absl::string_view uri,
                                int preferred_id,
                                RtpTransceiverDirection direction);
   ~RtpHeaderExtensionCapability();
@@ -251,8 +251,8 @@
 // RTP header extension, see RFC8285.
 struct RTC_EXPORT RtpExtension {
   RtpExtension();
-  RtpExtension(std::string uri, int id);
-  RtpExtension(std::string uri, int id, bool encrypt);
+  RtpExtension(absl::string_view uri, int id);
+  RtpExtension(absl::string_view uri, int id, bool encrypt);
   ~RtpExtension();
 
   std::string ToString() const;