Hookup the PpdProvider in the printer setup flow.

chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_SETTINGS_CHROMEOS_CUPS_PRINTERS_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_SETTINGS_CHROMEOS_CUPS_PRINTERS_HANDLER_H_
 
 #include <memory>
 #include <string>
 #include <vector>
@@ skipping 70 matching lines @@
                     int index,
                     void* params) override;
 
   void HandleStartDiscovery(const base::ListValue* args);
   void HandleStopDiscovery(const base::ListValue* args);
 
   // chromeos::PrinterDiscoverer::Observer override:
   void OnPrintersFound(const std::vector<Printer>& printers) override;
   void OnDiscoveryDone() override;
 
+  // Invokes debugd to add the printer to CUPS.  If |ipp_everywhere| is true,
+  // automatic configuration will be attempted.  |ppd_path| should be a path

[Carlson, 2016/12/01 22:59:52]

Comment should indicate that ppd_path is ignored (or should be empty) if
ipp_everywhere is true.

[skau, 2016/12/05 20:40:51]

Done.

+  // to the ppd cache appropriate for the printer.
+  void AddPrinterToCups(std::unique_ptr<Printer> printer,
+                        const std::string& ppd_path,

[Carlson, 2016/12/05 23:40:11]

Shouldn't this be a base::FilePath?

[skau, 2016/12/06 20:55:46]

It goes onto the DBus as a string so it doesn't really matter.  But FilePath is
safer.  I've changed it.

+                        bool ipp_everywhere);
+
+  // Callback for PpdProvider::ResolveCallback.
+  void OnPPDResolved(std::unique_ptr<Printer> printer,
+                     printing::PpdProvider::CallbackResultCode result,
+                     base::FilePath ppd_path);
+
   std::unique_ptr<chromeos::PrinterDiscoverer> printer_discoverer_;
   std::unique_ptr<chromeos::printing::PpdProvider> ppd_provider_;
 
   Profile* profile_;
   scoped_refptr<ui::SelectFileDialog> select_file_dialog_;
   std::string webui_callback_id_;
 
   base::WeakPtrFactory<CupsPrintersHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CupsPrintersHandler);
 };
 
 }  // namespace settings
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_UI_WEBUI_SETTINGS_CHROMEOS_CUPS_PRINTERS_HANDLER_H_