Added a diagnostic user message when enumerating printers fails. Also tweaked...

chrome/service/cloud_print/printer_job_handler.h

Index: chrome/service/cloud_print/printer_job_handler.h
===================================================================
--- chrome/service/cloud_print/printer_job_handler.h	(revision 72357)
+++ chrome/service/cloud_print/printer_job_handler.h	(working copy)
@@ -91,6 +91,10 @@
      virtual void OnPrinterJobHandlerShutdown(
         PrinterJobHandler* job_handler, const std::string& printer_id) = 0;
      virtual void OnAuthError() = 0;
+     // Called when the PrinterJobHandler cannot find the printer locally. The
+     // delegate returns true if the printer should be deleted from the server,
+     // false otherwise.
+     virtual bool OnPrinterNotFound(const std::string& printer_name) = 0;

[gene1, 2011/01/24 19:46:47]

OK, after reading the comment, I understand what OnPrinterNotFound() do :). Can
we rename function to be less confusing? Something like
HasCompletePrinterInfo(), or similar?

[sanjeevr, 2011/01/24 19:59:05]

But the check is not HasCompletePrinterInfo. The notification is as it stands.
The delegate may choose not to delete the printer from the server for other
reasons.

 
    protected:
      virtual ~Delegate() {}