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

Unified Diff: remoting/protocol/input_event_timestamps.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/ice_connection_to_client.cc ('k') | remoting/protocol/input_event_timestamps.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/input_event_timestamps.h
diff --git a/remoting/protocol/input_event_timestamps.h b/remoting/protocol/input_event_timestamps.h
new file mode 100644
index 0000000000000000000000000000000000000000..52bca48bd48e74268c14adc95a9e0b7f0dacea3e
--- /dev/null
+++ b/remoting/protocol/input_event_timestamps.h
@@ -0,0 +1,66 @@
+// Copyright 2016 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.
+
+#ifndef REMOTING_PROTOCOL_INPUT_EVENT_TIMESTAMPS_H_
+#define REMOTING_PROTOCOL_INPUT_EVENT_TIMESTAMPS_H_
+
+#include "base/memory/ref_counted.h"
+#include "base/time/time.h"
+
+namespace remoting {
+namespace protocol {
+
+// Used on the host side to track timestamps for input events.
+struct InputEventTimestamps {
+ // Client-side timestamps. This value comes from the client clock, so it
+ // should not be used for any calculations on the host side (except in tests).
+ base::TimeTicks client_timestamp;
+
+ // Time when the event was processed by the host.
+ base::TimeTicks host_timestamp;
+
+ bool is_null() { return client_timestamp.is_null(); }
+};
+
+// InputEventTimestampsSource is used by VideoStream implementations to get
+// event timestamps that are sent back to the client as part of VideoStats
+// message.
+class InputEventTimestampsSource
+ : public base::RefCountedThreadSafe<InputEventTimestampsSource> {
+ public:
+ InputEventTimestampsSource() {}
+
+ // Returns event timestamps for the input event that was received since the
+ // previous call. Null InputEventTimestamps value is returned if no input
+ // events were received. If multiple input events were received, then
+ // timestamps for the last one should be returned
+ virtual InputEventTimestamps TakeLastEventTimestamps() = 0;
+
+ protected:
+ friend base::RefCountedThreadSafe<InputEventTimestampsSource>;
+ virtual ~InputEventTimestampsSource() {}
+};
+
+// Simple implementations of InputEventTimestampsSource that just stores the
+// value provided to OnEventReceived().
+class InputEventTimestampsSourceImpl : public InputEventTimestampsSource {
+ public:
+ InputEventTimestampsSourceImpl();
+
+ void OnEventReceived(InputEventTimestamps timestamps);
+
+ // InputEventTimestampsSource implementation.
+ InputEventTimestamps TakeLastEventTimestamps() override;
+
+ protected:
+ ~InputEventTimestampsSourceImpl() override;
+
+ private:
+ InputEventTimestamps last_timestamps_;
+};
+
+} // namespace protocol
+} // namespace remoting
+
+#endif // REMOTING_PROTOCOL_INPUT_EVENT_TIMESTAMPS_H_
« no previous file with comments | « remoting/protocol/ice_connection_to_client.cc ('k') | remoting/protocol/input_event_timestamps.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698