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

Unified Diff: chrome/browser/ui/webui/print_preview/extension_printer_handler.cc

Issue 922833004: Make PrinterProviderAPI a pure interface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
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/ui/webui/print_preview/extension_printer_handler.cc
diff --git a/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc b/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
index e3eef1a1972d768dc444b171a4e13c78f82a3d7a..aaeb3ff6c344cb38f71e97236d14ad0a7bce3ede 100644
--- a/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
+++ b/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
@@ -19,6 +19,8 @@
#include "components/cloud_devices/common/cloud_device_description.h"
#include "components/cloud_devices/common/printer_description.h"
#include "extensions/browser/api/printer_provider/printer_provider_api.h"
+#include "extensions/browser/api/printer_provider/printer_provider_api_factory.h"
+#include "extensions/browser/api/printer_provider/printer_provider_print_job.h"
#include "printing/pdf_render_settings.h"
#include "printing/pwg_raster_settings.h"
@@ -87,8 +89,8 @@ void ExtensionPrinterHandler::Reset() {
void ExtensionPrinterHandler::StartGetPrinters(
const PrinterHandler::GetPrintersCallback& callback) {
- extensions::PrinterProviderAPI::GetFactoryInstance()
- ->Get(browser_context_)
+ extensions::PrinterProviderAPIFactory::GetInstance()
+ ->GetForBrowserContext(browser_context_)
->DispatchGetPrintersRequested(
base::Bind(&ExtensionPrinterHandler::WrapGetPrintersCallback,
weak_ptr_factory_.GetWeakPtr(), callback));
@@ -97,8 +99,8 @@ void ExtensionPrinterHandler::StartGetPrinters(
void ExtensionPrinterHandler::StartGetCapability(
const std::string& destination_id,
const PrinterHandler::GetCapabilityCallback& callback) {
- extensions::PrinterProviderAPI::GetFactoryInstance()
- ->Get(browser_context_)
+ extensions::PrinterProviderAPIFactory::GetInstance()
+ ->GetForBrowserContext(browser_context_)
->DispatchGetCapabilityRequested(
destination_id,
base::Bind(&ExtensionPrinterHandler::WrapGetCapabilityCallback,
@@ -112,8 +114,8 @@ void ExtensionPrinterHandler::StartPrint(
const gfx::Size& page_size,
const scoped_refptr<base::RefCountedMemory>& print_data,
const PrinterHandler::PrintCallback& callback) {
- scoped_ptr<extensions::PrinterProviderAPI::PrintJob> print_job(
- new extensions::PrinterProviderAPI::PrintJob());
+ scoped_ptr<extensions::PrinterProviderPrintJob> print_job(
+ new extensions::PrinterProviderPrintJob());
print_job->printer_id = destination_id;
print_job->ticket_json = ticket_json;
@@ -163,7 +165,7 @@ void ExtensionPrinterHandler::ConvertToPWGRaster(
void ExtensionPrinterHandler::DispatchPrintJob(
const PrinterHandler::PrintCallback& callback,
- scoped_ptr<extensions::PrinterProviderAPI::PrintJob> print_job,
+ scoped_ptr<extensions::PrinterProviderPrintJob> print_job,
const scoped_refptr<base::RefCountedMemory>& print_data) {
if (!print_data) {
WrapPrintCallback(callback, false, kInvalidDataPrintError);
@@ -172,8 +174,8 @@ void ExtensionPrinterHandler::DispatchPrintJob(
print_job->document_bytes = print_data;
- extensions::PrinterProviderAPI::GetFactoryInstance()
- ->Get(browser_context_)
+ extensions::PrinterProviderAPIFactory::GetInstance()
+ ->GetForBrowserContext(browser_context_)
->DispatchPrintRequested(
*print_job, base::Bind(&ExtensionPrinterHandler::WrapPrintCallback,
weak_ptr_factory_.GetWeakPtr(), callback));

Powered by Google App Engine
This is Rietveld 408576698