Bug 991273 - Warn instead of asserting until we can find a solution. a=sledru
authorPaul Adenot <paul@paul.cx>
Tue, 13 May 2014 13:27:21 +0200
changeset 200628 c5eabaa4b2662fc9337fb4a76c7e425ff48077cb
parent 200627 e3522dec2b847fb20ed5f1318e9377282329e5d7
child 200629 0b10831bb6a9cb1c65b223be804d5f2dcbeb59a1
push id486
push userasasaki@mozilla.com
push dateMon, 14 Jul 2014 18:39:42 +0000
treeherdermozilla-release@d33428174ff1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssledru
bugs991273
milestone31.0
Bug 991273 - Warn instead of asserting until we can find a solution. a=sledru
content/media/MediaSegment.h
--- a/content/media/MediaSegment.h
+++ b/content/media/MediaSegment.h
@@ -287,17 +287,17 @@ protected:
     }
     mChunks.MoveElementsFrom(aSource->mChunks);
   }
 
   void AppendSliceInternal(const MediaSegmentBase<C, Chunk>& aSource,
                            TrackTicks aStart, TrackTicks aEnd)
   {
     MOZ_ASSERT(aStart <= aEnd, "Endpoints inverted");
-    MOZ_ASSERT(aStart >= 0 && aEnd <= aSource.mDuration, "Slice out of range");
+    NS_WARN_IF_FALSE(aStart >= 0 && aEnd <= aSource.mDuration, "Slice out of range");
     mDuration += aEnd - aStart;
     TrackTicks offset = 0;
     for (uint32_t i = 0; i < aSource.mChunks.Length() && offset < aEnd; ++i) {
       const Chunk& c = aSource.mChunks[i];
       TrackTicks start = std::max(aStart, offset);
       TrackTicks nextOffset = offset + c.GetDuration();
       TrackTicks end = std::min(aEnd, nextOffset);
       if (start < end) {