Adding the ability to release hosts after testing to the app remoting test driver.

This change introduces a new class which will wrap calls to the
ReportIssue API.  This request will be used to tell the service when we
are done with the remote hosts we used for testing so that they can be
cleaned up and the test will receive a fresh instance the next time it
is launched.  I've also moved the logic to create the service urls out
into a separate class which can be used by framework and unit test
code.

Also updates the behavior for 'show-host-availability' so that it can be
used when running tests for debugging info purposes.

Also cleans up some minor cpplint errors.

BUG=

Review URL: https://codereview.chromium.org/1219063002

Cr-Commit-Position: refs/heads/master@{#337155}

1 files changed, 1 insertions, 22 deletions

diff --git a/remoting/test/remote_host_info_fetcher.h b/remoting/test/remote_host_info_fetcher.h
index d3af340..984e375 100644
--- a/remoting/test/remote_host_info_fetcher.h
+++ b/remoting/test/remote_host_info_fetcher.h
@@ -11,6 +11,7 @@
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "net/url_request/url_fetcher_delegate.h"
+#include "remoting/test/app_remoting_service_urls.h"
 #include "remoting/test/remote_host_info.h"
 
 namespace net {
@@ -23,28 +24,6 @@ class URLRequestContextGetter;
 namespace remoting {
 namespace test {
 
-// Used by the RemoteHostInfoFetcher to make HTTP requests and also by the
-// unittests for this class to set fake response data for these URLs.
-const char kDevServiceEnvironmentUrlFormat[] =
-    "https://www-googleapis-test.sandbox.google.com/appremoting/v1beta1_dev/"
-    "applications/%s/run";
-const char kTestServiceEnvironmentUrlFormat[] =
-    "https://www-googleapis-test.sandbox.google.com/appremoting/v1beta1/"
-    "applications/%s/run";
-const char kStagingServiceEnvironmentUrlFormat[] =
-    "https://www-googleapis-test.sandbox.google.com/appremoting/"
-    "v1beta1_staging/applications/%s/run";
-
-// Specifies the service API to call for app remoting host information.
-// Note: When adding new environments, add them before kUnknownEnvironment as
-//       the last entry is used for bounds checking.
-enum ServiceEnvironment {
-  kDeveloperEnvironment,
-  kTestingEnvironment,
-  kStagingEnvironment,
-  kUnknownEnvironment
-};
-
 // Supplied by the client for each remote host info request and returns a valid,
 // initialized RemoteHostInfo object on success.
 typedef base::Callback<void(const RemoteHostInfo& remote_host_info)>