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

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

Issue 2910513002: [CRD iOS] Fix telemetry issues (Closed)
Patch Set: Resolve feedback 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/ios/facade/host_info.h ('k') | remoting/ios/facade/remoting_service.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/ios/facade/host_info.cc
diff --git a/remoting/ios/facade/host_info.cc b/remoting/ios/facade/host_info.cc
index 6e14be4354ea58949b32cd6d9ccf1e466e764a1e..939fc504e66db2f2784ce337730a1b547d827da8 100644
--- a/remoting/ios/facade/host_info.cc
+++ b/remoting/ios/facade/host_info.cc
@@ -51,6 +51,21 @@ bool HostInfo::ParseHostInfo(const base::DictionaryValue& 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;
« no previous file with comments | « remoting/ios/facade/host_info.h ('k') | remoting/ios/facade/remoting_service.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698