Bug 1305826: P1. Fix move constructor. r?gerald draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Sat, 01 Oct 2016 12:02:46 +1000
changeset 419871 37de5be8219f7686966afa5dbaa59c96e018560d
parent 419870 86aa837ccb9af3a0abe69f787c81dd44da6c083d
child 419872 f4d27bc7e19995d330ed453cf5115880bec2d802
push id31032
push userbmo:jyavenard@mozilla.com
push dateSat, 01 Oct 2016 09:38:22 +0000
reviewersgerald
bugs1305826
milestone52.0a1
Bug 1305826: P1. Fix move constructor. r?gerald With a const input, the object would never be moved but copied instead MozReview-Commit-ID: 7VUJDLCxOQJ
dom/media/AudioConverter.h
--- a/dom/media/AudioConverter.h
+++ b/dom/media/AudioConverter.h
@@ -79,17 +79,17 @@ public:
                   "Conversion not implemented yet");
   }
   explicit AudioDataBuffer(AlignedShortBuffer&& aBuffer)
     : mBuffer(Move(aBuffer))
   {
     static_assert(Format == AudioConfig::FORMAT_S16,
                   "Conversion not implemented yet");
   }
-  explicit AudioDataBuffer(const AlignedFloatBuffer&& aBuffer)
+  explicit AudioDataBuffer(AlignedFloatBuffer&& aBuffer)
     : mBuffer(Move(aBuffer))
   {
     static_assert(Format == AudioConfig::FORMAT_FLT,
                   "Conversion not implemented yet");
   }
   AudioDataBuffer& operator=(AudioDataBuffer&& aOther)
   {
     mBuffer = Move(aOther.mBuffer);