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

Unified Diff: remoting/client/ios/host_preferences.mm

Issue 2809793002: Build Chrome Remote Desktop for iOS in Chromium trybots. (Closed)
Patch Set: Working around an unused variable issue. 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/host_preferences.mm
diff --git a/remoting/client/ios/host_preferences.mm b/remoting/client/ios/host_preferences.mm
index f229ce25a9426dfa9f5b389092fd4d654527b2da..1377cb91b7fd4c3db490b643dc4c6be4e1874a05 100644
--- a/remoting/client/ios/host_preferences.mm
+++ b/remoting/client/ios/host_preferences.mm
@@ -44,7 +44,9 @@ static NSString* const kHostPreferencesPairSecretKey = @"PairSecret";
NSError* keychainError =
remoting::ios::WriteHostPreferencesToKeychain(writeData);
- DLOG_IF(ERROR, !keychainError) << "Could not write to keychain.";
+ if (!keychainError) {
sdefresne 2017/04/12 15:27:52 This is https://codereview.chromium.org/2817513004
nicholss 2017/04/12 15:35:32 I am working on removing this file from this cl :)
+ LOG(ERROR) << "Could not write to keychain.";
+ }
}
+ (HostPreferences*)hostForId:(NSString*)hostId {
« media/media_options.gni ('K') | « remoting/BUILD.gn ('k') | remoting/remoting_enable.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698