Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(140)

Unified Diff: remoting/client/ios/facade/host_list_fetcher.h

Issue 2871993003: Moving the iOS directory to be remoting top level. (Closed)
Patch Set: //remoting/ios was the old landing target for the internal iOS application. Fix. Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/ios/facade/host_info.cc ('k') | remoting/client/ios/facade/host_list_fetcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/ios/facade/host_list_fetcher.h
diff --git a/remoting/client/ios/facade/host_list_fetcher.h b/remoting/client/ios/facade/host_list_fetcher.h
deleted file mode 100644
index af1f8dc44e2b7cbfffa541da84601544d26f056b..0000000000000000000000000000000000000000
--- a/remoting/client/ios/facade/host_list_fetcher.h
+++ /dev/null
@@ -1,69 +0,0 @@
-// 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.
-// TODO(nicholss): Consider moving this to an extern and conditionally include
-// prod or test environment urls based on config. A test env app would be nice.
-const char kHostListProdRequestUrl[] =
- "https://www.googleapis.com/chromoting/v1/@me/hosts";
-
-// 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_
« no previous file with comments | « remoting/client/ios/facade/host_info.cc ('k') | remoting/client/ios/facade/host_list_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698