Chromium Code Reviews| Index: chrome/browser/printing/printing_message_filter.cc |
| diff --git a/chrome/browser/printing/printing_message_filter.cc b/chrome/browser/printing/printing_message_filter.cc |
| index 1f9ad53afe2bfa15ce6dad911b71f61e49e7f13c..2191588c8a6663beaecb7d9b48860affafd1138b 100644 |
| --- a/chrome/browser/printing/printing_message_filter.cc |
| +++ b/chrome/browser/printing/printing_message_filter.cc |
| @@ -284,18 +284,15 @@ void PrintingMessageFilter::OnScriptedPrintReply( |
| } |
| void PrintingMessageFilter::OnUpdatePrintSettings( |
| - int document_cookie, const DictionaryValue& job_settings, |
| - IPC::Message* reply_msg) { |
| + const DictionaryValue& job_settings, IPC::Message* reply_msg) { |
| scoped_refptr<printing::PrinterQuery> printer_query; |
| - print_job_manager_->PopPrinterQuery(document_cookie, &printer_query); |
|
kmadhusu
2011/09/16 20:39:11
Validate the document_cookie and if it is valid we
arthurhsu
2011/09/16 23:39:34
Done.
|
| - if (printer_query.get()) { |
| - CancelableTask* task = NewRunnableMethod( |
| - this, |
| - &PrintingMessageFilter::OnUpdatePrintSettingsReply, |
| - printer_query, |
| - reply_msg); |
| - printer_query->SetSettings(job_settings, task); |
| - } |
| + printer_query = new printing::PrinterQuery(); |
| + CancelableTask* task = NewRunnableMethod( |
| + this, |
| + &PrintingMessageFilter::OnUpdatePrintSettingsReply, |
| + printer_query, |
| + reply_msg); |
| + printer_query->SetSettings(job_settings, task); |
| } |
| void PrintingMessageFilter::OnUpdatePrintSettingsReply( |