TabCapture: enforce active refresh if there is un-sampled compositor update

media/capture/content/video_capture_oracle.h

Index: media/capture/content/video_capture_oracle.h
diff --git a/media/capture/content/video_capture_oracle.h b/media/capture/content/video_capture_oracle.h
index 8d545e29a61c3ea16f0e462dc35b68c42f41d723..81038c948a8646f05d24ff601fab175de0ccdf6e 100644
--- a/media/capture/content/video_capture_oracle.h
+++ b/media/capture/content/video_capture_oracle.h
@@ -145,6 +145,16 @@ class CAPTURE_EXPORT VideoCaptureOracle {
   // sanity-check that event times are monotonically non-decreasing.
   base::TimeTicks last_event_time_[kNumEvents];
 
+  // Set to true if there have been updates to the source content that were not
+  // sampled. This will prevent passive refresh requests from being satisfied
+  // when an active refresh should be used instead.
+  bool source_is_dirty_;
+
+  // The last |event| passed to ObserveEventAndDecideCapture() where it returned
+  // a "yes" decision. This is used by RecordCapture() to decide whether the
+  // |source_is_dirty_| flag can be cleared.
+  Event last_event_causing_capture_;

[miu, 2017/03/24 21:17:28]

Per discussion, we don't need |last_event_causing_capture_| anymore (since a
RecordCapture() call always means the dirty flag can be cleared).

[braveyao, 2017/03/24 23:12:27]

Done.

+
   // Updated by the last call to ObserveEventAndDecideCapture() with the
   // estimated duration of the next frame to sample.  This is zero if the method
   // returned false.