commit ff61f3a55500cd8046e614dc54b08452ff296ea6
author Karl Wiberg <kwiberg@webrtc.org> Fri Feb 28 09:01:18 2020
committer Commit Bot <commit-bot@chromium.org> Fri Feb 28 09:26:11 2020
tree 878f60c7fab9a6a001f186e44ab5391f5107aacb
parent 603838356502a35141c8723baa97c1c566c3e87e

Fix + test copying of fixed-sized ArrayView rvalues

Previously, only lvalues were tested, and only lvalues worked.

Bug: webrtc:11389
Change-Id: I524e9d63e0840c3ba274dbe2062d78f72d79019d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/169347
Commit-Queue: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30644}

api/array_view.h

diff --git a/api/array_view.h b/api/array_view.h
index f7130dc..a66369a 100644
--- a/api/array_view.h
+++ b/api/array_view.h
@@ -213,6 +213,14 @@
       : ArrayView(u.data(), u.size()) {
     static_assert(U::size() == Size, "Sizes must match exactly");
   }
+  template <
+      typename U,
+      typename std::enable_if<Size != impl::kArrayViewVarSize &&
+                              HasDataAndSize<U, T>::value>::type* = nullptr>
+  ArrayView(const U& u)  // NOLINT(runtime/explicit)
+      : ArrayView(u.data(), u.size()) {
+    static_assert(U::size() == Size, "Sizes must match exactly");
+  }
 
   // (Only if size is variable.) Construct an ArrayView from any type U that
   // has a size() method whose return value converts implicitly to size_t, and