[CRD iOS] Consistent host list order and showing last online time

remoting/ios/facade/host_info.cc

 // 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/ios/facade/host_info.h"
 
 #include "base/logging.h"
 
 namespace remoting {
 
@@ skipping 33 matching lines @@
   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("hostOS", &host_os)) {
-    LOG(ERROR) << "hostOS was not found in host_info";
-    return false;
-  }
-
-  if (!host_info.GetString("hostOsVersion", &host_os_version)) {
-    LOG(ERROR) << "hostOsVersion was not found in host_info";
-    return false;
-  }
-
-  if (!host_info.GetString("hostVersion", &host_version)) {
-    LOG(ERROR) << "hostVersion 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;
   }
 
+  std::string updated_time_iso;
+  if (host_info.GetString("updatedTime", &updated_time_iso)) {
+    if (!base::Time::FromString(updated_time_iso.c_str(), &updated_time)) {
+      LOG(WARNING) << "Failed to parse updatedTime";
+    }
+  }
+
+  host_info.GetString("hostOs", &host_os);
+  host_info.GetString("hostOsVersion", &host_os_version);
+  host_info.GetString("hostVersion", &host_version);
+
   host_info.GetString("hostOfflineReason", &offline_reason);
 
   return true;
 }
 
 bool HostInfo::IsReadyForConnection() const {
   return !host_jid.empty() && status == kHostStatusOnline;
 }
 
 }  // namespace remoting