Remote assistance on Chrome OS Part IX - Input injection on Ozone

remoting/host/display_rotation_filter.h

Index: remoting/host/display_rotation_filter.h
diff --git a/remoting/host/display_rotation_filter.h b/remoting/host/display_rotation_filter.h
new file mode 100644
index 0000000000000000000000000000000000000000..a9172ae06b0067f8951c5329db6b1942a4c7bc2b
--- /dev/null
+++ b/remoting/host/display_rotation_filter.h
@@ -0,0 +1,21 @@
+// Copyright 2014 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_HOST_DISPLAY_ROTATION_FILTER_H_
+#define REMOTING_HOST_DISPLAY_ROTATION_FILTER_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "remoting/protocol/input_filter.h"
+
+namespace remoting {
+
+// Transform the mouse location of the event to inject based on the display
+// rotation settings.  Currently, Ozone is the only platform that requires
+// custom transformation.

[Wez, 2014/11/26 02:28:34]

This comment should describe the method's function, e.g:

"Returns an InputFilter that rotates input event coordinates appropriately for
the current display rotation settings."

[kelvinp, 2014/12/02 00:12:26]

Done.

+scoped_ptr<protocol::InputFilter> CreateDisplayRotationFilter(

[Wez, 2014/11/26 02:28:34]

This is an InputFilter that rotates the coordinates of input events, so let's
call it RotatingInputFilter or InputRotator, i.e. CreateRotatingInputFilter
here()?

[kelvinp, 2014/12/02 00:12:26]

Done.

+    protocol::InputStub* input_stub);
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_DISPLAY_ROTATION_FILTER_H_