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

Unified Diff: remoting/test/host_info.cc

Issue 1212333011: Retrieved Hostlist information from Directory service. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added missing offline_host check. Created 5 years, 5 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/test/host_info.h ('k') | remoting/test/host_list_fetcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/test/host_info.cc
diff --git a/remoting/test/host_info.cc b/remoting/test/host_info.cc
new file mode 100644
index 0000000000000000000000000000000000000000..341829f662e5f40815c61b2b29e817de77cc8172
--- /dev/null
+++ b/remoting/test/host_info.cc
@@ -0,0 +1,74 @@
+// Copyright 2015 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.
+
+#include "remoting/test/host_info.h"
+
+#include "base/logging.h"
+
+namespace remoting {
+namespace test {
+
+HostInfo::HostInfo() {
+}
+
+HostInfo::~HostInfo() {
+}
+
+bool HostInfo::ParseHostInfo(const base::DictionaryValue& host_info) {
+ const base::ListValue* list_value = nullptr;
+
+ // Add TokenUrlPatterns to HostInfo.
+ if (host_info.GetList("tokenUrlPatterns", &list_value)) {
+ if (!list_value->empty()) {
+ for (base::Value* item : *list_value) {
+ std::string token_url_pattern;
+ if (!item->GetAsString(&token_url_pattern)) {
+ return false;
+ }
+ token_url_patterns.push_back(token_url_pattern);
+ }
+ }
+ }
+
+ std::string response_status;
+ host_info.GetString("status", &response_status);
+ if (response_status == "ONLINE") {
+ status = kHostStatusOnline;
+ } else if (response_status == "OFFLINE") {
+ status = kHostStatusOffline;
+ } else {
+ LOG(ERROR) << "Response Status is " << response_status;
+ return false;
+ }
+
+ if (!host_info.GetString("hostId", &host_id)) {
+ LOG(ERROR) << "hostId was not found in host_info";
+ return false;
+ }
+
+ if (!host_info.GetString("hostName", &host_name)) {
+ LOG(ERROR) << "hostName was not found in host_info";
+ return false;
+ }
+
+ if (!host_info.GetString("publicKey", &public_key)) {
+ LOG(ERROR) << "publicKey was not found for " << host_name;
+ return false;
+ }
+
+ // If the host entry was created but the host was never online, then the jid
+ // is never set.
+ if (!host_info.GetString("jabberId", &host_jid) &&
+ status == kHostStatusOnline) {
+ LOG(ERROR) << host_name << " is online but is missing a jabberId";
+ return false;
+ }
+
+ host_info.GetString("hostOfflineReason", &offline_reason);
+
+ return true;
+}
+
+} // namespace test
+} // namespace remoting
« no previous file with comments | « remoting/test/host_info.h ('k') | remoting/test/host_list_fetcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698