Support for connecting to localhost on the chromoting test driver.

remoting/remoting_test.gypi

Index: remoting/remoting_test.gypi
diff --git a/remoting/remoting_test.gypi b/remoting/remoting_test.gypi
index 73a9f17448e9a03babdcf19310f88eb8e52c8032..d3bd3964e53afe8acf9af7992b9bb4ee8c905e8e 100644
--- a/remoting/remoting_test.gypi
+++ b/remoting/remoting_test.gypi
@@ -120,9 +120,13 @@
       'sources': [
         'test/access_token_fetcher.cc',
         'test/access_token_fetcher.h',
+        'test/host_info.cc',
+        'test/host_info.h',

[joedow, 2015/07/14 19:58:44]

Why were the chromoting and app remoting specific files moved to the common
target?  I don't think this is necessary...

[tonychun, 2015/07/15 17:32:50]

I have made a new struct called ConnectionInfo which takes away the need handle
different types of hosts (from app remoting and chromoting). They weren't very
different to begin with, so this was relatively simple.

         'test/refresh_token_store.cc',
         'test/refresh_token_store.h',
         'test/remote_connection_observer.h',
+        'test/remote_host_info.cc',
+        'test/remote_host_info.h',
         'test/test_chromoting_client.cc',
         'test/test_chromoting_client.h',
         'test/test_video_renderer.cc',
@@ -134,17 +138,16 @@
       'target_name': 'chromoting_test_driver',
       'type': '<(gtest_target_type)',
       'dependencies': [
+        '../remoting/proto/chromotocol.gyp:chromotocol_proto_lib',

[joedow, 2015/07/14 19:58:44]

Shouldn't this go in the common section since it is shared by both apps?

[tonychun, 2015/07/15 17:32:50]

I tried moving this dependency up to 'remoting_test_driver_common', but for some
odd reason, neither driver builds if I do this. Perhaps we can go over this to
find some work around.

[anandc, 2015/07/15 21:12:29]

How about moving it to remoting_test_common?

         'remoting_test_driver_common',
       ],
       'defines': [
         'VERSION=<(version_full)',
       ],
       'sources': [
-        'test/host_info.cc',
-        'test/host_info.h',
+        'test/chromoting_test_driver.cc',
         'test/host_list_fetcher.cc',
         'test/host_list_fetcher.h',
-        'test/chromoting_test_driver.cc',
       ],
       'include_dirs': [
         '../testing/gtest/include',
@@ -175,8 +178,6 @@
         'test/app_remoting_test_driver_environment.cc',
         'test/app_remoting_test_driver_environment.h',
         'test/remote_application_details.h',
-        'test/remote_host_info.cc',
-        'test/remote_host_info.h',
         'test/remote_host_info_fetcher.cc',
         'test/remote_host_info_fetcher.h',
       ],