Include USB printers in printer list as "provisional" devices.

chrome/browser/ui/webui/print_preview/print_preview_handler.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_HANDLER_H_
 
 #include <string>
 
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/printing/print_view_manager_observer.h"
 #include "components/signin/core/browser/gaia_cookie_manager_service.h"
 #include "content/public/browser/web_ui_message_handler.h"
 #include "ui/shell_dialogs/select_file_dialog.h"
 
 #if defined(ENABLE_SERVICE_DISCOVERY)
 #include "chrome/browser/local_discovery/privet_local_printer_lister.h"
 #include "chrome/browser/local_discovery/service_discovery_shared_client.h"
 #endif  // ENABLE_SERVICE_DISCOVERY
 
+class ExtensionPrinterHandler;
 class PrinterHandler;
 class PrintPreviewUI;
 class PrintSystemTaskProxy;
 
 namespace base {
 class DictionaryValue;
 class RefCountedBytes;
 }
 
 namespace content {
@@ skipping 81 matching lines @@
 
   static bool PrivetPrintingEnabled();
 
   content::WebContents* preview_web_contents() const;
 
   PrintPreviewUI* print_preview_ui() const;
 
   // Gets the list of printers. |args| is unused.
   void HandleGetPrinters(const base::ListValue* args);
 
-  // Starts getting all local privet printers. |arg| is unused.
+  // Starts getting all local privet printers. |args| is unused.
   void HandleGetPrivetPrinters(const base::ListValue* args);
 
-  // Starts getting all local extension managed printers. |arg| is unused.
+  // Starts getting all local extension managed printers. |args| is unused.
   void HandleGetExtensionPrinters(const base::ListValue* args);
 
+  // Starts getting all local USB printers supported by extensions. |args| is
+  // unused.
+  void HandleGetExtensionUsbPrinters(const base::ListValue* args);
+
   // Stops getting all local privet printers. |arg| is unused.
   void HandleStopGetPrivetPrinters(const base::ListValue* args);
 
   // Asks the initiator renderer to generate a preview.  First element of |args|
   // is a job settings JSON string.
   void HandleGetPreview(const base::ListValue* args);
 
   // Gets the job settings from Web UI and initiate printing. First element of
   // |args| is a job settings JSON string.
   void HandlePrint(const base::ListValue* args);
@@ skipping 160 matching lines @@
   // Lazily creates |extension_printer_handler_| that can be used to handle
   // extension printers requests.
   void EnsureExtensionPrinterHandlerSet();
 
   // Called when a list of printers is reported by an extension.
   // |printers|: The list of printers managed by the extension.
   // |done|: Whether all the extensions have reported the list of printers
   //     they manage.
   void OnGotPrintersForExtension(const base::ListValue& printers, bool done);
 
+  // Called with the list of USB printers supported by extensions.
+  // |printers|: The list of printers supported by extensions.
+  void OnGotExtensionUsbPrinters(const base::ListValue& printers);
+
   // Called when an extension reports the set of print capabilites for a
   // printer.
   // |printer_id|: The id of the printer whose capabilities are reported.
   // |capabilities|: The printer capabilities.
   void OnGotExtensionPrinterCapabilities(
       const std::string& printer_id,
       const base::DictionaryValue& capabilities);
 
   // Called when an extension print job is completed.
   // |success|: Whether the job succeeded.
@@ skipping 44 matching lines @@
   scoped_ptr<local_discovery::PrivetHTTPResolution> privet_http_resolution_;
   scoped_ptr<local_discovery::PrivetV1HTTPClient> privet_http_client_;
   scoped_ptr<local_discovery::PrivetJSONOperation>
       privet_capabilities_operation_;
   scoped_ptr<local_discovery::PrivetLocalPrintOperation>
       privet_local_print_operation_;
 #endif
 
   // Handles requests for extension printers. Created lazily by calling
   // |EnsureExtensionPrinterHandlerSet|.
-  scoped_ptr<PrinterHandler> extension_printer_handler_;
+  scoped_ptr<ExtensionPrinterHandler> extension_printer_handler_;

[Vitaly Buka (NO REVIEWS), 2015/05/27 16:58:40]

Idea was to have common interface for different print system.
So here should be PrinterHandler.

[Reilly Grant (use Gerrit), 2015/05/28 21:45:00]

Done.

 
   // Notifies tests that want to know if the PDF has been saved. This doesn't
   // notify the test if it was a successful save, only that it was attempted.
   base::Closure pdf_file_saved_closure_;
 
   base::WeakPtrFactory<PrintPreviewHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintPreviewHandler);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_HANDLER_H_