Update SourceBufferStream and its unit tests to always expect valid durations.

media/filters/source_buffer_stream.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/filters/source_buffer_stream.h"
 
 #include <algorithm>
 #include <map>
 
 #include "base/bind.h"
@@ skipping 699 matching lines @@
 bool SourceBufferStream::IsMonotonicallyIncreasing(
     const BufferQueue& buffers) const {
   DCHECK(!buffers.empty());
   base::TimeDelta prev_timestamp = last_appended_buffer_timestamp_;
   bool prev_is_keyframe = last_appended_buffer_is_keyframe_;
   for (BufferQueue::const_iterator itr = buffers.begin();
        itr != buffers.end(); ++itr) {
     base::TimeDelta current_timestamp = (*itr)->GetDecodeTimestamp();
     bool current_is_keyframe = (*itr)->IsKeyframe();
     DCHECK(current_timestamp != kNoTimestamp());
+    DCHECK((*itr)->duration() >= base::TimeDelta())
+        << "Packet with invalid duration."
+        << " pts " << (*itr)->timestamp().InSecondsF()
+        << " dts " << (*itr)->GetDecodeTimestamp().InSecondsF()
+        << " dur " << (*itr)->duration().InSecondsF();
 
     if (prev_timestamp != kNoTimestamp()) {
       if (current_timestamp < prev_timestamp) {
         MEDIA_LOG(log_cb_) << "Buffers were not monotonically increasing.";
         return false;
       }
 
       if (current_timestamp == prev_timestamp &&
           !AllowSameTimestamp(prev_is_keyframe, current_is_keyframe,
                               GetType())) {
@@ skipping 1532 matching lines @@
     return false;
 
   DCHECK_NE(have_splice_buffers, have_preroll_buffer);
   splice_buffers_index_ = 0;
   pending_buffer_.swap(*out_buffer);
   pending_buffers_complete_ = false;
   return true;
 }
 
 }  // namespace media