Win Video Capture: Sending the timestamps provided by the driver to the capture pipeline

media/capture/video/win/sink_input_pin_win.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/capture/video/win/sink_input_pin_win.h"
 
 #include <cstring>
 
 // Avoid including strsafe.h via dshow as it will cause build warnings.
 #define NO_DSHOW_STRSAFE
@@ skipping 164 matching lines @@
   uint8* buffer = NULL;
 
   if (length <= 0) {
     DLOG(WARNING) << "Media sample length is 0 or less.";
     return S_FALSE;
   }
 
   if (FAILED(sample->GetPointer(&buffer)))
     return S_FALSE;
 
-  observer_->FrameReceived(buffer, length);
+  REFERENCE_TIME start_time, end_time;
+
+  sample->GetTime(&start_time, &end_time);
+

[mcasas, 2015/09/09 23:33:26]

I deeply distrust all this code. Would you mind adding
a DCHECK to certify that |start_time| <= |end_time| 
and |start_time| is non negative?

[qiangchen, 2015/09/10 18:03:01]

Done.

Added a fallback to old code path if the driver does not provide a timestamp.

+  observer_->FrameReceived(buffer, length, start_time);
   return S_OK;
 }
 
 SinkInputPin::~SinkInputPin() {
 }
 
 }  // namespace media