Moving input related classes to the input folder.

remoting/client/BUILD.gn

 # 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.
 
 static_library("client") {
   # Disabled the source filters because there are _mac files that need to
   # be compiled on all platforms.
   set_sources_assignment_filter([])
 
   sources = [
     "audio_player.cc",
     "audio_player.h",
     "chromoting_client.cc",
     "chromoting_client.h",
     "chromoting_session.cc",
     "chromoting_session.h",
     "client_context.cc",
     "client_context.h",
     "client_telemetry_logger.cc",
     "client_telemetry_logger.h",
     "client_user_interface.h",
     "connect_to_host_info.cc",
     "connect_to_host_info.h",
     "cursor_shape_stub_proxy.cc",
     "cursor_shape_stub_proxy.h",
     "dual_buffer_frame_consumer.cc",
     "dual_buffer_frame_consumer.h",
     "empty_cursor_filter.cc",
     "empty_cursor_filter.h",
+    "gesture_interpreter.cc",
+    "gesture_interpreter.h",
     "host_experiment_sender.cc",
     "host_experiment_sender.h",
     "key_event_mapper.cc",
     "key_event_mapper.h",
-    "native_device_keymap.cc",
-    "native_device_keymap.h",
     "normalizing_input_filter_cros.cc",
     "normalizing_input_filter_cros.h",
     "normalizing_input_filter_mac.cc",
     "normalizing_input_filter_mac.h",
     "normalizing_input_filter_win.cc",
     "normalizing_input_filter_win.h",
     "queued_task_poster.cc",
     "queued_task_poster.h",
     "server_log_entry_client.cc",
     "server_log_entry_client.h",
     "software_video_renderer.cc",
     "software_video_renderer.h",
-    "touch_input_scaler.cc",
-    "touch_input_scaler.h",
   ]
   set_sources_assignment_filter(sources_assignment_filter)
 
   configs += [
     "//build/config/compiler:wexit_time_destructors",
     "//remoting/build/config:version",
     "//remoting/build/config:enable_webrtc_remoting_client",
   ]
 
   deps = [
     "//remoting/base",
     "//remoting/client/input",

[Yuwei, 2017/05/23 21:28:26]

You may need //remoting/client/ui here to use RendererProxy.

Also given that RendererProxy is only used by GestureInterpreter and platform
specific code, you may just move it to client/ and move the client/display
dependency from client/ui to client/.

[nicholss, 2017/05/23 22:01:23]

I added //remoting/client/display:proxy, see how that feels. I am not 100% sure
yet.

[Yuwei, 2017/05/23 22:20:42]

TBH I don't feel its good to have a new target to just wrap the RendererProxy...

Turns out you do want /client to depend on /client/ui so that GestureInterpreter
can use FlingAnimation. That's the reason the Android trybot failed...

Then I think I'm fine with keeping RendererProxy in /client/ui

[nicholss, 2017/05/23 22:52:23]

I moved it to flat display. I don't think it is a UI component, it is part of
the renderer component in my mind.

     "//remoting/codec:decoder",
     "//remoting/protocol",
     "//third_party/libyuv",
     "//third_party/webrtc/modules/desktop_capture:primitives",
     "//ui/events:dom_keycode_converter",
   ]
 
   libs = []
 
   if (is_nacl) {
     sources -= [
       "client_telemetry_logger.cc",
       "server_log_entry_client.cc",
     ]
   } else {
     sources += [
       "chromoting_client_runtime.cc",
       "chromoting_client_runtime.h",
     ]
   }
   if (is_android) {
     sources += [
       "audio_player_android.cc",
       "audio_player_android.h",
-      "native_device_keymap_android.cc",
     ]
-    sources -= [ "native_device_keymap.cc" ]
     libs += [
       "android",
       "OpenSLES",
     ]
   }
-  if (is_ios) {
-    sources += [ "native_device_keymap_ios.cc" ]
-    sources -= [ "native_device_keymap.cc" ]
-  }
 }
 
 source_set("unit_tests") {
   testonly = true
 
   # Disabled the source filters because there are _mac files that need to
   # be compiled on all platforms.
   set_sources_assignment_filter([])
   sources = [
     "audio_player_unittest.cc",
     "chromoting_client_runtime_unittest.cc",
     "client_telemetry_logger_unittest.cc",
     "dual_buffer_frame_consumer_unittest.cc",
     "empty_cursor_filter_unittest.cc",
     "key_event_mapper_unittest.cc",
     "normalizing_input_filter_cros_unittest.cc",
     "normalizing_input_filter_mac_unittest.cc",
     "normalizing_input_filter_win_unittest.cc",
     "queued_task_poster_unittest.cc",
     "server_log_entry_client_unittest.cc",
     "software_video_renderer_unittest.cc",
-    "touch_input_scaler_unittest.cc",
   ]
   set_sources_assignment_filter(sources_assignment_filter)
 
   configs += [
     "//remoting/build/config:version",
     "//remoting/build/config:enable_webrtc_remoting_client",
   ]
 
   deps = [
     ":client",
     "//remoting/proto",
     "//testing/gmock",
     "//testing/gtest",
     "//third_party/webrtc/base:rtc_base_approved",
   ]
 }