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

Side by Side Diff: remoting/test/host_list_fetcher.cc

Issue 2542343004: Updating the Chromoting Test Driver to target the test environment (Closed)
Patch Set: Addressing feedback Created 4 years 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 unified diff | Download patch
« no previous file with comments | « remoting/test/host_list_fetcher.h ('k') | remoting/test/host_list_fetcher_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "remoting/test/host_list_fetcher.h" 5 #include "remoting/test/host_list_fetcher.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/callback_helpers.h" 8 #include "base/callback_helpers.h"
9 #include "base/json/json_reader.h" 9 #include "base/json/json_reader.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
11 #include "base/threading/thread_task_runner_handle.h" 11 #include "base/threading/thread_task_runner_handle.h"
12 #include "base/values.h" 12 #include "base/values.h"
13 #include "net/http/http_status_code.h" 13 #include "net/http/http_status_code.h"
14 #include "net/url_request/url_fetcher.h" 14 #include "net/url_request/url_fetcher.h"
15 #include "remoting/base/url_request_context_getter.h" 15 #include "remoting/base/url_request_context_getter.h"
16 16
17 namespace remoting { 17 namespace remoting {
18 namespace test { 18 namespace test {
19 19
20 HostListFetcher::HostListFetcher() { 20 HostListFetcher::HostListFetcher() {
21 } 21 }
22 22
23 HostListFetcher::~HostListFetcher() { 23 HostListFetcher::~HostListFetcher() {
24 } 24 }
25 25
26 void HostListFetcher::RetrieveHostlist( 26 void HostListFetcher::RetrieveHostlist(const std::string& access_token,
27 const std::string& access_token, 27 const std::string& target_url,
28 const HostlistCallback& callback) { 28 const HostlistCallback& callback) {
29
30 VLOG(2) << "HostListFetcher::RetrieveHostlist() called"; 29 VLOG(2) << "HostListFetcher::RetrieveHostlist() called";
31 30
32 DCHECK(!access_token.empty()); 31 DCHECK(!access_token.empty());
33 DCHECK(!callback.is_null()); 32 DCHECK(!callback.is_null());
34 DCHECK(hostlist_callback_.is_null()); 33 DCHECK(hostlist_callback_.is_null());
35 34
36 hostlist_callback_ = callback; 35 hostlist_callback_ = callback;
37 36
38 request_context_getter_ = new remoting::URLRequestContextGetter( 37 request_context_getter_ = new remoting::URLRequestContextGetter(
39 base::ThreadTaskRunnerHandle::Get(), // network_runner 38 /*network_runner=*/base::ThreadTaskRunnerHandle::Get(),
40 base::ThreadTaskRunnerHandle::Get()); // file_runner 39 /*file_runner=*/base::ThreadTaskRunnerHandle::Get());
41 40
42 request_ = net::URLFetcher::Create( 41 request_ =
43 GURL(kHostListProdRequestUrl), net::URLFetcher::GET, this); 42 net::URLFetcher::Create(GURL(target_url), net::URLFetcher::GET, this);
44 request_->SetRequestContext(request_context_getter_.get()); 43 request_->SetRequestContext(request_context_getter_.get());
45 request_->AddExtraRequestHeader("Authorization: OAuth " + access_token); 44 request_->AddExtraRequestHeader("Authorization: OAuth " + access_token);
46 request_->Start(); 45 request_->Start();
47 } 46 }
48 47
49 bool HostListFetcher::ProcessResponse( 48 bool HostListFetcher::ProcessResponse(
50 std::vector<HostInfo>* hostlist) { 49 std::vector<HostInfo>* hostlist) {
51 int response_code = request_->GetResponseCode(); 50 int response_code = request_->GetResponseCode();
52 if (response_code != net::HTTP_OK) { 51 if (response_code != net::HTTP_OK) {
53 LOG(ERROR) << "Hostlist request failed with error code: " << response_code; 52 LOG(ERROR) << "Hostlist request failed with error code: " << response_code;
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 std::vector<HostInfo> hostlist; 105 std::vector<HostInfo> hostlist;
107 106
108 if (!ProcessResponse(&hostlist)) { 107 if (!ProcessResponse(&hostlist)) {
109 hostlist.clear(); 108 hostlist.clear();
110 } 109 }
111 base::ResetAndReturn(&hostlist_callback_).Run(hostlist); 110 base::ResetAndReturn(&hostlist_callback_).Run(hostlist);
112 } 111 }
113 112
114 } // namespace test 113 } // namespace test
115 } // namespace remoting 114 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/test/host_list_fetcher.h ('k') | remoting/test/host_list_fetcher_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698