[CRD iOS] Send key events to the session.

remoting/client/chromoting_session.h

 // Copyright 2013 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_CLIENT_CHROMOTING_SESSION_H_
 #define REMOTING_CLIENT_CHROMOTING_SESSION_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 97 matching lines @@
 
   // Moves the host's cursor to the specified coordinates, optionally with some
   // mouse button depressed. If |button| is BUTTON_UNDEFINED, no click is made.
   void SendMouseEvent(int x,
                       int y,
                       protocol::MouseEvent_MouseButton button,
                       bool button_down);
   void SendMouseWheelEvent(int delta_x, int delta_y);
 
   // Sends the provided keyboard scan code to the host.
+  // TODO(nicholss): sending scan code here is really weird.
+  // Can we fix this up so scan_code != 0 is it's own api endpoint?
   bool SendKeyEvent(int scan_code, int key_code, bool key_down);
 
+  // Sends the usb key code to the host.
+  bool SendKeyEvent(int usb_key_code, bool key_down);

[Yuwei, 2017/05/15 19:38:06]

FYI it's not recommended to overload functions in Chromium. It could also be
confusing to the reader when they read both SendKeyEvent(int, int, int) and
SendKeyEvent(int, int) and had hard time realizing the difference between native
keycode and usb keycode O_o

Not sure what's the best name though. SendKeyEventWithUsbkeycode?

[nicholss, 2017/05/15 23:06:10]

Acknowledged.

+
+  // Send utf8 encoded text to the host.
   void SendTextEvent(const std::string& text);
 
   // Sends the provided touch event payload to the host.
   void SendTouchEvent(const protocol::TouchEvent& touch_event);
 
   // Enables or disables the video channel. May be called from any thread.
   void EnableVideoChannel(bool enable);
 
   void SendClientMessage(const std::string& type, const std::string& data);
 
@@ skipping 86 matching lines @@
 
   base::WeakPtr<ChromotingSession> weak_ptr_;
   base::WeakPtrFactory<ChromotingSession> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingSession);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_CHROMOTING_SESSION_H_