MSE: Test append window discontinuity detection

media/filters/frame_processor.cc

 // Copyright 2014 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/frame_processor.h"
 
 #include "base/stl_util.h"
 #include "media/base/buffers.h"
 #include "media/base/stream_parser_buffer.h"
 
@@ skipping 108 matching lines @@
     if (decode_timestamp > presentation_timestamp) {
       // TODO(wolenetz): Determine whether DTS>PTS should really be allowed. See
       // http://crbug.com/354518.
       DVLOG(2) << __FUNCTION__ << ": WARNING: Frame DTS("
                << decode_timestamp.InSecondsF() << ") > PTS("
                << presentation_timestamp.InSecondsF() << ")";
     }
 
     // TODO(acolwell/wolenetz): All stream parsers must emit valid (positive)
     // frame durations. For now, we allow non-negative frame duration.
+    // Until frame durations are trusted, SourceBufferStream inter-buffer
+    // distance fudging may merge otherwise discontinuous ranges.
     // See http://crbug.com/351166.
     if (frame_duration == kNoTimestamp()) {
       DVLOG(2) << __FUNCTION__ << ": Frame missing duration (kNoTimestamp())";
       return false;
     }
     if (frame_duration <  base::TimeDelta()) {
       DVLOG(2) << __FUNCTION__ << ": Frame duration negative: "
                << frame_duration.InSecondsF();
       return false;
     }
@@ skipping 230 matching lines @@
 }
 
 void FrameProcessor::SetAllTrackBuffersNeedRandomAccessPoint() {
   for (TrackBufferMap::iterator itr = track_buffers_.begin();
        itr != track_buffers_.end(); ++itr) {
     itr->second->set_needs_random_access_point(true);
   }
 }
 
 }  // namespace media