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

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

Issue 948413006: Move callback and host port args into Start method (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@no_sdc
Patch Set: Created 5 years, 10 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_local_printer_lister.cc
diff --git a/chrome/browser/local_discovery/privet_local_printer_lister.cc b/chrome/browser/local_discovery/privet_local_printer_lister.cc
index ef5ebc26a2b5a0b8677ecabca7312116b6aef589..b2460ad9095e896a203f4220fe56b069eb73c777 100644
--- a/chrome/browser/local_discovery/privet_local_printer_lister.cc
+++ b/chrome/browser/local_discovery/privet_local_printer_lister.cc
@@ -63,14 +63,12 @@ void PrivetLocalPrinterLister::DeviceChanged(
linked_ptr<DeviceContext> context(new DeviceContext);
context->has_local_printing = false;
context->description = description;
- context->privet_resolution = privet_http_factory_->CreatePrivetHTTP(
- name,
+ context->privet_resolution = privet_http_factory_->CreatePrivetHTTP(name);
+ device_contexts_[name] = context;
+ context->privet_resolution->Start(
description.address,
base::Bind(&PrivetLocalPrinterLister::OnPrivetResolved,
base::Unretained(this), name));
-
- device_contexts_[name] = context;
- context->privet_resolution->Start();
}
}

Powered by Google App Engine
This is Rietveld 408576698