Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1119)

Unified Diff: remoting/protocol/video_frame_pump.h

Issue 2413553003: Add InputEventTimestampSource interface. (Closed)
Patch Set: msvc compilation Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/protocol/protocol_mock_objects.h ('k') | remoting/protocol/video_frame_pump.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/video_frame_pump.h
diff --git a/remoting/protocol/video_frame_pump.h b/remoting/protocol/video_frame_pump.h
index 5483400c8f9294b53fe39c2427b86b728dd652c2..6fe57e84bed2aab0b94340d0a08010762a1c296f 100644
--- a/remoting/protocol/video_frame_pump.h
+++ b/remoting/protocol/video_frame_pump.h
@@ -80,8 +80,9 @@ class VideoFramePump : public VideoStream,
~VideoFramePump() override;
// VideoStream interface.
+ void SetEventTimestampsSource(scoped_refptr<InputEventTimestampsSource>
+ event_timestamps_source) override;
void Pause(bool pause) override;
- void OnInputEventReceived(int64_t event_timestamp) override;
void SetLosslessEncode(bool want_lossless) override;
void SetLosslessColor(bool want_lossless) override;
void SetObserver(Observer* observer) override;
@@ -95,12 +96,9 @@ class VideoFramePump : public VideoStream,
FrameTimestamps();
~FrameTimestamps();
- // The following two fields are set only for one frame after each incoming
- // input event. |input_event_client_timestamp| is event timestamp
- // received from the client. |input_event_received_time| is local time when
- // the event was received.
- int64_t input_event_client_timestamp = -1;
- base::TimeTicks input_event_received_time;
+ // The following field is not-null for a single frame after each incoming
+ // input event.
+ InputEventTimestamps input_event_timestamps;
base::TimeTicks capture_started_time;
base::TimeTicks capture_ended_time;
@@ -159,6 +157,8 @@ class VideoFramePump : public VideoStream,
// Used to encode captured frames. Always accessed on the encode thread.
std::unique_ptr<VideoEncoder> encoder_;
+ scoped_refptr<InputEventTimestampsSource> event_timestamps_source_;
+
// Interface through which video frames are passed to the client.
protocol::VideoStub* video_stub_;
@@ -174,9 +174,6 @@ class VideoFramePump : public VideoStream,
// captured.
CaptureScheduler capture_scheduler_;
- // Timestamps for the frame to be captured next.
- std::unique_ptr<FrameTimestamps> next_frame_timestamps_;
-
// Timestamps for the frame that's being captured.
std::unique_ptr<FrameTimestamps> captured_frame_timestamps_;
« no previous file with comments | « remoting/protocol/protocol_mock_objects.h ('k') | remoting/protocol/video_frame_pump.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698