Interrogate PpdProvider from PrintPreview.

chrome/browser/ui/webui/print_preview/printer_backend_proxy.cc

 // Copyright 2016 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/ui/webui/print_preview/printer_backend_proxy.h"
 
 #include <string>
 #include <utility>
 
 #include "base/logging.h"
@@ skipping 34 matching lines @@
   }
 
   PrinterBasicInfo basic_info;
   if (!print_backend->GetPrinterBasicInfo(device_name, &basic_info)) {
     return nullptr;
   }
 
   return GetSettingsOnBlockingPool(device_name, basic_info);
 }
 
+class PrinterBackendProxyImpl : public PrinterBackendProxy {

[Carlson, 2016/12/02 19:10:22]

Maybe a better suffix here would be Chrome instead of Impl?  More orthogonal to
the ChromeOS version.  Also a short comment about what this class is would be
good.

[skau, 2016/12/05 23:13:34]

Renamed to 'Default'.  Chrome seems odd since ChromeOS isn't orthogonal to
Chrome.  ChromeOS is just special.

+ public:
+  std::string GetDefaultPrinterOnBlockingPoolThread() override {
+    DCHECK(
+        content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
+
+    scoped_refptr<printing::PrintBackend> print_backend(
+        PrintBackend::CreateInstance(nullptr));
+
+    std::string default_printer = print_backend->GetDefaultPrinterName();
+    VLOG(1) << "Default Printer: " << default_printer;
+    return default_printer;
+  }
+
+  void EnumeratePrinters(const EnumeratePrintersCallback& cb) override {
+    base::PostTaskAndReplyWithResult(
+        content::BrowserThread::GetBlockingPool(), FROM_HERE,
+        base::Bind(&EnumeratePrintersOnBlockingPoolThread), cb);
+  }
+
+  void ConfigurePrinterAndFetchCapabilities(
+      const std::string& device_name,
+      const PrinterSetupCallback& cb) override {
+    DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+
+    base::PostTaskAndReplyWithResult(
+        content::BrowserThread::GetBlockingPool(), FROM_HERE,
+        base::Bind(&FetchCapabilitiesOnBlockingPool, device_name), cb);
+  }
+};
+
 }  // namespace
 
-std::string GetDefaultPrinterOnBlockingPoolThread() {
-  DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
-
-  scoped_refptr<printing::PrintBackend> print_backend(
-      PrintBackend::CreateInstance(nullptr));
-
-  std::string default_printer = print_backend->GetDefaultPrinterName();
-  VLOG(1) << "Default Printer: " << default_printer;
-  return default_printer;
-}
-
-void EnumeratePrinters(Profile* /* profile */,
-                       const EnumeratePrintersCallback& cb) {
-  base::PostTaskAndReplyWithResult(
-      content::BrowserThread::GetBlockingPool(), FROM_HERE,
-      base::Bind(&EnumeratePrintersOnBlockingPoolThread), cb);
-}
-
-void ConfigurePrinterAndFetchCapabilities(Profile* /* profile */,
-                                          const std::string& device_name,
-                                          const PrinterSetupCallback& cb) {
-  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-
-  base::PostTaskAndReplyWithResult(
-      content::BrowserThread::GetBlockingPool(), FROM_HERE,
-      base::Bind(&FetchCapabilitiesOnBlockingPool, device_name), cb);
+std::unique_ptr<PrinterBackendProxy> PrinterBackendProxy::Create() {
+  return base::MakeUnique<PrinterBackendProxyImpl>();
 }
 
 }  // namespace printing