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

Unified Diff: chrome/browser/local_discovery/privet_device_resolver.cc

Issue 861523002: Updated parsing of PrivetV3 mDns record. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Thu 01/22/2015 10:42:57.49 Created 5 years, 11 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: chrome/browser/local_discovery/privet_device_resolver.cc
diff --git a/chrome/browser/local_discovery/privet_device_resolver.cc b/chrome/browser/local_discovery/privet_device_resolver.cc
index 6bbb13b6bf5b31dd93b44b509103c84f862eed0d..915d42c8f7f51e6ee9a7d5916b2fa1e1258b9590 100644
--- a/chrome/browser/local_discovery/privet_device_resolver.cc
+++ b/chrome/browser/local_discovery/privet_device_resolver.cc
@@ -29,14 +29,12 @@ void PrivetDeviceResolver::Start() {
void PrivetDeviceResolver::OnServiceResolved(
ServiceResolver::RequestStatus request_status,
const ServiceDescription& service_description) {
- DeviceDescription device_description;
if (request_status != ServiceResolver::STATUS_SUCCESS) {
- callback_.Run(false, device_description);
+ callback_.Run(false, DeviceDescription());
return;
}
- device_description.FillFromServiceDescription(service_description);
- callback_.Run(true, device_description);
+ callback_.Run(true, DeviceDescription(service_description));
}
} // namespace local_discovery

Powered by Google App Engine
This is Rietveld 408576698