Add media size capability to PDF printer.

chrome/browser/printing/print_job_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/printing/print_job_manager.h"
 
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/printing/print_job.h"
 #include "chrome/browser/printing/printer_query.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 39 matching lines @@
 }
 
 scoped_refptr<PrinterQuery> PrintQueriesQueue::CreatePrinterQuery() {
   scoped_refptr<PrinterQuery> job = new printing::PrinterQuery;
   base::AutoLock lock(lock_);
   job->SetWorkerDestination(destination_);
   return job;
 }
 
 void PrintQueriesQueue::Shutdown() {
-  base::AutoLock lock(lock_);
-  queued_queries_.clear();
-  destination_ = NULL;
+  PrinterQueries queries_to_stop;
+  {
+    base::AutoLock lock(lock_);
+    queued_queries_.swap(queries_to_stop);
+    destination_ = NULL;
+  }
+  for (PrinterQueries::iterator itr = queries_to_stop.begin();
+       itr != queries_to_stop.end(); ++itr) {

[Vitaly Buka (NO REVIEWS), 2014/05/29 00:58:34]

iterating using for (size_t i = 0) is shorter

+    scoped_refptr<printing::PrinterQuery> query_to_stop(*itr);

[Vitaly Buka (NO REVIEWS), 2014/05/29 00:58:34]

you don't need temp var query_to_stop

+    query_to_stop->message_loop()->PostTask(
+        FROM_HERE, base::Bind(&PrinterQuery::StopWorker, query_to_stop));
+  }
 }
 
 PrintJobManager::PrintJobManager() : is_shutdown_(false) {
   registrar_.Add(this, chrome::NOTIFICATION_PRINT_JOB_EVENT,
                  content::NotificationService::AllSources());
 }
 
 PrintJobManager::~PrintJobManager() {
 }
 
@@ skipping 77 matching lines @@
       break;
     }
     default: {
       NOTREACHED();
       break;
     }
   }
 }
 
 }  // namespace printing