commit 9cc5fffee150b3160714de2c9b07ad3c28728d8e
author Niels Möller <nisse@webrtc.org> Fri Feb 11 08:30:26 2022
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Feb 11 11:31:54 2022
tree b57ddfadf1322fbb31c472f8313a65be11cb1056
parent b4e06d032e6f82a65c52ed0c5364ae9e7c0a0215

Convert a few more uses of rtc::split to use string_view

Bug: webrtc:13579
Change-Id: I84bdb908bf390924c6d67cd1c5aabcc9e62f33da
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251581
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35976}

media/engine/webrtc_voice_engine.cc

diff --git a/media/engine/webrtc_voice_engine.cc b/media/engine/webrtc_voice_engine.cc
index 69fd238..b7b0ad7 100644
--- a/media/engine/webrtc_voice_engine.cc
+++ b/media/engine/webrtc_voice_engine.cc
@@ -1666,8 +1666,8 @@
     RTC_LOG(LS_WARNING) << "audio/RED missing fmtp parameters.";
     return false;
   }
-  std::vector<std::string> redundant_payloads;
-  rtc::split(red_parameters->second, '/', &redundant_payloads);
+  std::vector<absl::string_view> redundant_payloads =
+      rtc::split(red_parameters->second, '/');
   // 32 is chosen as a maximum upper bound for consistency with the
   // red payload splitter.
   if (redundant_payloads.size() < 2 || redundant_payloads.size() > 32) {

modules/desktop_capture/linux/wayland/egl_dmabuf.cc

diff --git a/modules/desktop_capture/linux/wayland/egl_dmabuf.cc b/modules/desktop_capture/linux/wayland/egl_dmabuf.cc
index a4de19d..088d1c4 100644
--- a/modules/desktop_capture/linux/wayland/egl_dmabuf.cc
+++ b/modules/desktop_capture/linux/wayland/egl_dmabuf.cc
@@ -327,9 +327,8 @@
     return;
   }
 
-  std::vector<std::string> client_extensions_no_display;
-  rtc::split(client_extensions_cstring_no_display, ' ',
-             &client_extensions_no_display);
+  std::vector<absl::string_view> client_extensions_no_display =
+      rtc::split(client_extensions_cstring_no_display, ' ');
   for (const auto& extension : client_extensions_no_display) {
     egl_.extensions.push_back(std::string(extension));
   }
@@ -388,8 +387,8 @@
       EglQueryString(egl_.display, EGL_EXTENSIONS);
   client_extensions_string = client_extensions_cstring_display;
 
-  std::vector<std::string> client_extensions;
-  rtc::split(client_extensions_string, ' ', &client_extensions);
+  std::vector<absl::string_view> client_extensions =
+      rtc::split(client_extensions_string, ' ');
   for (const auto& extension : client_extensions) {
     egl_.extensions.push_back(std::string(extension));
   }