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

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

Issue 2957943002: [CRD iOS] Add the host setup instruction page (Closed)
Patch Set: Add comment Created 3 years, 6 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/ios/facade/host_list_fetcher.cc
diff --git a/remoting/ios/facade/host_list_fetcher.cc b/remoting/ios/facade/host_list_fetcher.cc
index 469cbf31b88d113ed85c61042a48d1507644ceca..ea3340ef8c97b0c5e84ae9e5d9f36657917be1ba 100644
--- a/remoting/ios/facade/host_list_fetcher.cc
+++ b/remoting/ios/facade/host_list_fetcher.cc
@@ -104,8 +104,8 @@ bool HostListFetcher::ProcessResponse(
const base::ListValue* hosts = nullptr;
if (!data->GetList("items", &hosts)) {
- LOG(ERROR) << "Failed to find hosts in Hostlist response data";
- return false;
+ // This will happen if the user has no host.
+ return true;
}
// Any host_info with malformed data will not be added to the hostlist.

Powered by Google App Engine
This is Rietveld 408576698