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

Unified Diff: chrome/browser/printing/printer_query.cc

Issue 480303002: Use document from preview for System Dialog printing on Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tue 08/19/2014 11:17:57.84 Created 6 years, 4 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/printing/printer_query.cc
diff --git a/chrome/browser/printing/printer_query.cc b/chrome/browser/printing/printer_query.cc
index 42684a14daf9753d4201d520b5080fb69eb0db26..2020364ff193832050837021f0cc08d7d7ef9ec3 100644
--- a/chrome/browser/printing/printer_query.cc
+++ b/chrome/browser/printing/printer_query.cc
@@ -88,14 +88,17 @@ void PrinterQuery::GetSettings(
margin_type));
}
-void PrinterQuery::SetSettings(const base::DictionaryValue& new_settings,
- const base::Closure& callback) {
+void PrinterQuery::SetSettings(
+ scoped_ptr<PrintingUIWebContentsObserver> web_contents_observer,
+ scoped_ptr<base::DictionaryValue> new_settings,
+ const base::Closure& callback) {
StartWorker(callback);
worker_->PostTask(FROM_HERE,
base::Bind(&PrintJobWorker::SetSettings,
base::Unretained(worker_.get()),
- new_settings.DeepCopy()));
+ base::Passed(&web_contents_observer),
+ base::Passed(&new_settings)));
}
void PrinterQuery::SetWorkerDestination(

Powered by Google App Engine
This is Rietveld 408576698