CRD iOS: Updating remoting service to use oauth and host list fetcher.

remoting/client/ios/facade/host_list_fetcher.h

+// Copyright 2017 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_IOS_FACADE_HOST_LIST_FETCHER_H_
+#define REMOTING_CLIENT_IOS_FACADE_HOST_LIST_FETCHER_H_
+
+#include <memory>
+#include <string>
+#include <vector>
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "net/url_request/url_fetcher_delegate.h"
+#include "net/url_request/url_request_context_getter.h"
+#include "remoting/client/ios/facade/host_info.h"
+
+namespace remoting {
+
+// Used by the HostlistFetcher to make HTTP requests and also by the
+// unittests for this class to set fake response data for these URLs.
+const char kHostListProdRequestUrl[] =
+    "https://www.googleapis.com/"
+    "chromoting/v1/@me/hosts";

[joedow, 2017/04/05 22:30:55]

I don't think this line needs to be broken, looks like a git cl format quirk

[nicholss, 2017/04/07 18:16:15]

git cl format!!!

fixed.

+const char kHostListTestRequestUrl[] =
+    "https://www-googleapis-test.sandbox.google.com/chromoting/v1/@me/hosts";

[joedow, 2017/04/05 22:30:55]

Are you using this?  I'd remove it if you don't intend to target the test
environment.

[nicholss, 2017/04/07 18:16:15]

I will remove this but I am going to add a todo to remind that we might want to
make a test version of the app to connect to test envs.

+
+// Requests a host list from the directory service for an access token.
+// Destroying the RemoteHostInfoFetcher while a request is outstanding
+// will cancel the request. It is safe to delete the fetcher from within a
+// completion callback.  Must be used from a thread running a message loop.
+// The public method is virtual to allow for mocking and fakes.
+class HostListFetcher : public net::URLFetcherDelegate {
+ public:
+  HostListFetcher(const scoped_refptr<net::URLRequestContextGetter>&
+                      url_request_context_getter);
+  ~HostListFetcher() override;
+
+  // Supplied by the client for each hostlist request and returns a valid,
+  // initialized Hostlist object on success.
+  typedef base::Callback<void(const std::vector<remoting::HostInfo>& hostlist)>
+      HostlistCallback;
+
+  // Makes a service call to retrieve a hostlist. The
+  // callback will be called once the HTTP request has completed.
+  virtual void RetrieveHostlist(const std::string& access_token,
+                                const HostlistCallback& callback);
+
+ private:
+  // Processes the response from the directory service.
+  bool ProcessResponse(std::vector<remoting::HostInfo>* hostlist);
+
+  // net::URLFetcherDelegate interface.
+  void OnURLFetchComplete(const net::URLFetcher* source) override;
+
+  scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
+
+  // Holds the URLFetcher for the Host List request.
+  std::unique_ptr<net::URLFetcher> request_;
+
+  // Caller-supplied callback used to return hostlist on success.
+  HostlistCallback hostlist_callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(HostListFetcher);
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_CLIENT_IOS_FACADE_HOST_LIST_FETCHER_H_