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

media/capture/video/win/sink_input_pin_win.cc

Index: media/capture/video/win/sink_input_pin_win.cc
diff --git a/media/capture/video/win/sink_input_pin_win.cc b/media/capture/video/win/sink_input_pin_win.cc
index 9c515a3422e722790a7cfc051d8ff9150e91bdaf..50c3c99cebaff1adf8ed318410eaaf364eb8e80a 100644
--- a/media/capture/video/win/sink_input_pin_win.cc
+++ b/media/capture/video/win/sink_input_pin_win.cc
@@ -182,7 +182,16 @@ HRESULT SinkInputPin::Receive(IMediaSample* sample) {
   if (FAILED(sample->GetPointer(&buffer)))
     return S_FALSE;
 
-  observer_->FrameReceived(buffer, length);
+  REFERENCE_TIME start_time, end_time;
+

[mcasas, 2015/09/10 20:03:20]

no blank line

[qiangchen, 2015/09/10 21:37:45]

Done.

+  if (FAILED(sample->GetTime(&start_time, &end_time))) {
+    start_time = -1;
+    end_time = -1;

[mcasas, 2015/09/10 20:03:20]

Hmmm, the assignment and the use of |start_time| are a bit far now 
and that dilutes your original intention.

I suggest you change the interface signature of FrameReceived() to
 
void FrameReceived(const uint8* , int , base::TimeTicks)

and do all the calculations here:

REFERENCE_TIME start_time, end_time;
base::TimeTicks capture_timestamp;
if (SUCCEEDED(sample->GetTime(&start_time, &end_time))) {
  DCHECK_LE(start_time, end_time);
  capture_timestamp += base::TimeDelta::FromMicroseconds(start_time / 10);
} else {
  capture_timestamp = base::TimeTicks::Now();
}
observer_->FrameReceived(buffer, length, capture_timestamp);

Like this is clearer what is sent as timestamp and why.
WDYT?

[qiangchen, 2015/09/10 21:37:45]

Done.

+  }
+
+  DCHECK(start_time <= end_time);
+
+  observer_->FrameReceived(buffer, length, start_time);
   return S_OK;
 }