Make PrinterProviderAPI a pure interface

extensions/browser/api/printer_provider/printer_provider_apitest.cc

 // Copyright 2015 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 "base/bind.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.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 "extensions/browser/extension_registry.h"
 #include "extensions/common/extension.h"
 #include "extensions/shell/test/shell_apitest.h"
 #include "extensions/test/extension_test_message_listener.h"
 #include "extensions/test/result_catcher.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 using extensions::PrinterProviderAPI;
+using extensions::PrinterProviderAPIFactory;
 
 // Callback for PrinterProviderAPI::DispatchGetPrintersRequested calls.
 // It appends items in |printers| to |*printers_out|. If |done| is set, it runs
 // |callback|.
 void AppendPrintersAndRunCallbackIfDone(base::ListValue* printers_out,
                                         const base::Closure& callback,
                                         const base::ListValue& printers,
                                         bool done) {
   for (size_t i = 0; i < printers.GetSize(); ++i) {
     const base::DictionaryValue* printer = NULL;
@@ skipping 31 matching lines @@
 }
 
 // Tests for chrome.printerProvider API.
 class PrinterProviderApiTest : public extensions::ShellApiTest {
  public:
   PrinterProviderApiTest() {}
   ~PrinterProviderApiTest() override {}
 
   void StartGetPrintersRequest(
       const PrinterProviderAPI::GetPrintersCallback& callback) {
-    PrinterProviderAPI::GetFactoryInstance()
-        ->Get(browser_context())
+    PrinterProviderAPIFactory::GetInstance()
+        ->GetForBrowserContext(browser_context())
         ->DispatchGetPrintersRequested(callback);
   }
 
   void StartPrintRequest(const std::string& extension_id,
                          const PrinterProviderAPI::PrintCallback& callback) {
-    PrinterProviderAPI::PrintJob job;
+    extensions::PrinterProviderPrintJob job;
     job.printer_id = extension_id + ":printer_id";
     job.ticket_json = "{}";
     job.content_type = "content_type";
     const unsigned char kDocumentBytes[] = {'b', 'y', 't', 'e', 's', 0};
     job.document_bytes =
         new base::RefCountedBytes(kDocumentBytes, arraysize(kDocumentBytes));
 
-    PrinterProviderAPI::GetFactoryInstance()
-        ->Get(browser_context())
+    PrinterProviderAPIFactory::GetInstance()
+        ->GetForBrowserContext(browser_context())
         ->DispatchPrintRequested(job, callback);
   }
 
   void StartCapabilityRequest(
       const std::string& extension_id,
       const PrinterProviderAPI::GetCapabilityCallback& callback) {
-    PrinterProviderAPI::GetFactoryInstance()
-        ->Get(browser_context())
+    PrinterProviderAPIFactory::GetInstance()
+        ->GetForBrowserContext(browser_context())
         ->DispatchGetCapabilityRequested(extension_id + ":printer_id",
                                          callback);
   }
 
   // Loads chrome.printerProvider test app and initializes is for test
   // |test_param|.
   // When the app's background page is loaded, the app will send 'loaded'
   // message. As a response to the message it will expect string message
   // specifying the test that should be run. When the app initializes its state
   // (e.g. registers listener for a chrome.printerProvider event) it will send
@@ skipping 538 matching lines @@
                                      &printers, run_loop.QuitClosure()));
 
   ASSERT_TRUE(catcher.GetNextResult()) << catcher.message();
 
   run_loop.Run();
 
   EXPECT_TRUE(printers.empty());
 }
 
 }  // namespace