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

Unified Diff: remoting/ios/domain/host_info.mm

Issue 2912293003: [CRD iOS] Consistent host list order and showing last online time (Closed)
Patch Set: Fix comment 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/domain/host_info.h ('k') | remoting/ios/facade/host_info.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/ios/domain/host_info.mm
diff --git a/remoting/ios/domain/host_info.mm b/remoting/ios/domain/host_info.mm
index e0482370b05bd64544999e564f9b2bed8cffbf91..21f55d1dac25be6402271e1792c6632cbba4a129 100644
--- a/remoting/ios/domain/host_info.mm
+++ b/remoting/ios/domain/host_info.mm
@@ -56,12 +56,12 @@
host.publicKey = [svr objectForKey:@"publicKey"];
host.status = [svr objectForKey:@"status"];
- NSString* ISO8601DateString = [svr objectForKey:@"updatedTime"];
- if (ISO8601DateString != nil) {
- NSDateFormatter* dateFormatter = [[NSDateFormatter alloc] init];
- [dateFormatter setDateFormat:@"yyyy-MM-dd'T'HH:mm:ss.SSSz"];
- host.updatedTime = [dateFormatter dateFromString:ISO8601DateString];
- }
+ // NSString* ISO8601DateString = [svr objectForKey:@"updatedTime"];
nicholss 2017/05/31 19:32:37 Do we need this anymore then? Can we delete?
Yuwei 2017/05/31 20:36:39 I don't think it's needed any more. I'll delete it
+ // if (ISO8601DateString != nil) {
+ // NSDateFormatter* dateFormatter = [[NSDateFormatter alloc] init];
+ // [dateFormatter setDateFormat:@"yyyy-MM-dd'T'HH:mm:ss.SSSz"];
+ // host.updatedTime = [dateFormatter dateFromString:ISO8601DateString];
+ // }
[hostList addObject:host];
}
« no previous file with comments | « remoting/ios/domain/host_info.h ('k') | remoting/ios/facade/host_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698