3 files changed, 12 insertions, 14 deletions
diff --git a/netwerk/ipc/DocumentLoadListener.cpp b/netwerk/ipc/DocumentLoadListener.cpp
index 715e00c056..54c9a53e65 100644
--- a/netwerk/ipc/DocumentLoadListener.cpp
+++ b/netwerk/ipc/DocumentLoadListener.cpp
@@ -1439,7 +1439,7 @@ bool DocumentLoadListener::ResumeSuspendedChannel(
     streamListenerFunctions.Clear();
   }
 
-  ForwardStreamListenerFunctions(streamListenerFunctions, aListener);
+  ForwardStreamListenerFunctions(std::move(streamListenerFunctions), aListener);
 
   // We don't expect to get new stream listener functions added
   // via re-entrancy. If this ever happens, we should understand
@@ -2642,7 +2642,7 @@ DocumentLoadListener::OnDataAvailable(nsIRequest* aRequest,
 
   mStreamListenerFunctions.AppendElement(StreamListenerFunction{
       VariantIndex<1>{},
-      OnDataAvailableParams{aRequest, data, aOffset, aCount}});
+      OnDataAvailableParams{aRequest, std::move(data), aOffset, aCount}});
 
   return NS_OK;
 }
diff --git a/netwerk/ipc/NeckoCommon.cpp b/netwerk/ipc/NeckoCommon.cpp
index c08faa5747..9ad4483d32 100644
--- a/netwerk/ipc/NeckoCommon.cpp
+++ b/netwerk/ipc/NeckoCommon.cpp
@@ -14,28 +14,26 @@
 
 namespace mozilla::net {
 
-nsresult ForwardStreamListenerFunctions(
-    nsTArray<StreamListenerFunction>& aCalls, nsIStreamListener* aParent) {
+nsresult ForwardStreamListenerFunctions(nsTArray<StreamListenerFunction> aCalls,
+                                        nsIStreamListener* aParent) {
   nsresult rv = NS_OK;
   for (auto& variant : aCalls) {
     variant.match(
-        [&](const OnStartRequestParams& aParams) {
+        [&](OnStartRequestParams& aParams) {
           rv = aParent->OnStartRequest(aParams.request);
           if (NS_FAILED(rv)) {
             aParams.request->Cancel(rv);
           }
         },
-        [&](const OnDataAvailableParams& aParams) {
+        [&](OnDataAvailableParams& aParams) {
           // Don't deliver OnDataAvailable if we've
           // already failed.
           if (NS_FAILED(rv)) {
             return;
           }
           nsCOMPtr<nsIInputStream> stringStream;
-          rv = NS_NewByteInputStream(
-              getter_AddRefs(stringStream),
-              Span<const char>(aParams.data.get(), aParams.count),
-              NS_ASSIGNMENT_DEPEND);
+          rv = NS_NewCStringInputStream(getter_AddRefs(stringStream),
+                                        std::move(aParams.data));
           if (NS_SUCCEEDED(rv)) {
             rv = aParent->OnDataAvailable(aParams.request, stringStream,
                                           aParams.offset, aParams.count);
@@ -44,7 +42,7 @@ nsresult ForwardStreamListenerFunctions(
             aParams.request->Cancel(rv);
           }
         },
-        [&](const OnStopRequestParams& aParams) {
+        [&](OnStopRequestParams& aParams) {
           if (NS_SUCCEEDED(rv)) {
             aParent->OnStopRequest(aParams.request, aParams.status);
           } else {
@@ -52,7 +50,7 @@ nsresult ForwardStreamListenerFunctions(
           }
           rv = NS_OK;
         },
-        [&](const OnAfterLastPartParams& aParams) {
+        [&](OnAfterLastPartParams& aParams) {
           nsCOMPtr<nsIMultiPartChannelListener> multiListener =
               do_QueryInterface(aParent);
           if (multiListener) {
diff --git a/netwerk/ipc/NeckoCommon.h b/netwerk/ipc/NeckoCommon.h
index 6f63ef1b2a..57f770db68 100644
--- a/netwerk/ipc/NeckoCommon.h
+++ b/netwerk/ipc/NeckoCommon.h
@@ -135,8 +135,8 @@ using StreamListenerFunction =
     mozilla::Variant<OnStartRequestParams, OnDataAvailableParams,
                      OnStopRequestParams, OnAfterLastPartParams>;
 
-nsresult ForwardStreamListenerFunctions(
-    nsTArray<StreamListenerFunction>& aCalls, nsIStreamListener* aParent);
+nsresult ForwardStreamListenerFunctions(nsTArray<StreamListenerFunction> aCalls,
+                                        nsIStreamListener* aParent);
 
 }  // namespace net
 }  // namespace mozilla