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

Unified Diff: remoting/client/ios/facade/host_info.cc

Issue 2794013005: CRD iOS: Updating remoting service to use oauth and host list fetcher. (Closed)
Patch Set: Updated based on feedback. Created 3 years, 8 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
Index: remoting/client/ios/facade/host_info.cc
diff --git a/remoting/client/ios/facade/host_info.cc b/remoting/client/ios/facade/host_info.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e1284b650c38325b2ac2b238a98a473710ee95b5
--- /dev/null
+++ b/remoting/client/ios/facade/host_info.cc
@@ -0,0 +1,75 @@
+// 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.
+
+#include "remoting/client/ios/facade/host_info.h"
+
+#include "base/logging.h"
+
+namespace remoting {
+
+HostInfo::HostInfo() {}
+HostInfo::HostInfo(const HostInfo& other) = default;
+
+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 (const auto& 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;
+}
+
+bool HostInfo::IsReadyForConnection() const {
+ return !host_jid.empty() && status == kHostStatusOnline;
+}
+
+} // namespace remoting

Powered by Google App Engine
This is Rietveld 408576698