Code to send diagnostic messages about cloud print proxy. Currently diagnostic...

Code to send diagnostic messages about cloud print proxy. Currently diagnostic messages are sent if the print system fails to initialize (this can happen on Windows if the XPS Framework is not installed on XP) as well when printer registration fails because we could not get the capabilities for the printer.

BUG=None
TEST=None

Review URL: http://codereview.chromium.org/6245005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72037 0039d316-1c4b-4281-b951-d872f2087c98

chrome/app/generated_resources.grd

@@ -5364,6 +5364,12 @@ Keep your key file in a safe place. You will need it to create new versions of y
       <message name="IDS_CLOUD_PRINT_SETUP_TEST_PAGE" desc="Label on button for printing a test page.">
         Print a Test Page
       </message>
+      <message name="IDS_CLOUD_PRINT_REGISTER_PRINTER_FAILED" desc="Status message to be sent to the server when we failed to retrieve information about a printer to register.">
+        An error occurred while retrieving printer capabilities for printer <ph name="PRINTER_NAME">$1<ex>HP Laserjet 7900</ex></ph>. This printer could not be registered with <ph name="CLOUD_PRINT_NAME">Google Cloud Print</ph>.
+      </message>
+      <message name="IDS_CLOUD_PRINT_XPS_UNAVAILABLE" desc="Status message to be sent to the Cloud Print server when the XPS framework is missing.">
+        The Microsoft XML Paper Specification Essentials Pack is not installed on the machine running the Google Cloud Print connector.
+      </message>
 
       <!-- Print Preview -->
       <message name="IDS_PRINT_PREVIEW_TITLE" desc="Title for print preview page ">

chrome/browser/service/service_process_control.cc

@@ -4,6 +4,7 @@
 
 #include "chrome/browser/service/service_process_control.h"
 
+#include "app/app_switches.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/process_util.h"
@@ -215,6 +216,9 @@ void ServiceProcessControl::Launch(Task* success_task, Task* failure_task) {
     cmd_line->AppendSwitch(switches::kWaitForDebugger);
   }
 
+  std::string locale = g_browser_process->GetApplicationLocale();
+  cmd_line->AppendSwitchASCII(switches::kLang, locale);
+
   // And then start the process asynchronously.
   launcher_ = new Launcher(this, cmd_line);
   launcher_->Run(

chrome/chrome.gyp

@@ -1081,6 +1081,7 @@
       'type': '<(library)',
       'msvs_guid': '2DA87614-55C5-4E56-A17E-0CD099786197',
       'dependencies': [
+        'chrome_strings',
         'common',
         'common_net',
         '../base/base.gyp:base',

chrome/service/DEPS

 include_rules = [
+  "+grit",  # For generated headers
   # For Chromoting Host Process
   "+remoting/base",
   "+remoting/host",

chrome/service/cloud_print/cloud_print_consts.cc

@@ -14,6 +14,7 @@ const char kPrinterDefaultsValue[] = "defaults";
 const char kPrinterStatusValue[] = "status";
 const char kPrinterTagValue[] = "tag";
 const char kPrinterRemoveTagValue[] = "remove_tag";
+const char kMessageTextValue[] = "message";
 
 // Values in the respone JSON from the cloud print server
 const char kPrinterListValue[] = "printers";
@@ -53,3 +54,7 @@ const char kJobFetchReasonNotified[] = "notified";
 // Job fetch after a successful print to query for more jobs.
 const char kJobFetchReasonQueryMore[] = "querymore";
 
+// Short message ids for diagnostic messages sent to the server.
+const char kPrintSystemFailedMessageId[] = "printsystemfail";
+const char kGetPrinterCapsFailedMessageId[] = "getprncapsfail";
+

chrome/service/cloud_print/cloud_print_consts.h

@@ -17,6 +17,7 @@ extern const char kPrinterDefaultsValue[];
 extern const char kPrinterStatusValue[];
 extern const char kPrinterTagValue[];
 extern const char kPrinterRemoveTagValue[];
+extern const char kMessageTextValue[];
 // Values in the respone JSON from the cloud print server
 extern const char kPrinterListValue[];
 extern const char kSuccessValue[];
@@ -42,6 +43,8 @@ extern const char kJobFetchReasonStartup[];
 extern const char kJobFetchReasonPoll[];
 extern const char kJobFetchReasonNotified[];
 extern const char kJobFetchReasonQueryMore[];
+extern const char kPrintSystemFailedMessageId[];
+extern const char kGetPrinterCapsFailedMessageId[];
 
 // Max retry count for job data fetch requests.
 const int kJobDataMaxRetryCount = 5;

chrome/service/cloud_print/cloud_print_helpers.cc

@@ -137,6 +137,16 @@ GURL CloudPrintHelpers::GetUrlForJobStatusUpdate(
   return cloud_print_server_url.ReplaceComponents(replacements);
 }
 
+GURL CloudPrintHelpers::GetUrlForUserMessage(const GURL& cloud_print_server_url,
+                                             const std::string& message_id) {
+  std::string path(AppendPathToUrl(cloud_print_server_url, "user/message"));
+  GURL::Replacements replacements;
+  replacements.SetPathStr(path);
+  std::string query = StringPrintf("code=%s", message_id.c_str());
+  replacements.SetQueryStr(query);
+  return cloud_print_server_url.ReplaceComponents(replacements);
+}
+
 bool CloudPrintHelpers::ParseResponseJSON(
     const std::string& response_data, bool* succeeded,
     DictionaryValue** response_dict) {

chrome/service/cloud_print/cloud_print_helpers.h

@@ -35,6 +35,9 @@ class CloudPrintHelpers {
   static GURL GetUrlForJobStatusUpdate(
       const GURL& cloud_print_server_url, const std::string& job_id,
       const cloud_print::PrintJobDetails& details);
+  static GURL GetUrlForUserMessage(const GURL& cloud_print_server_url,
+                                   const std::string& message_id);
+
   // Parses the response data for any cloud print server request. The method
   // returns false if there was an error in parsing the JSON. The succeeded
   // value returns the value of the "success" value in the response JSON.

chrome/service/cloud_print/cloud_print_helpers_unittest.cc

@@ -73,6 +73,12 @@ void CheckURLs(const GURL& server_base_url) {
                               "code=2&message=Out%%20of%%20Paper&numpages=345&"
                               "pagesprinted=47", expected_url_base.c_str());
   EXPECT_EQ(expected_url, url.spec());
+
+  url = CloudPrintHelpers::GetUrlForUserMessage(server_base_url,
+                                                "blahmessageid");
+  expected_url = StringPrintf("%suser/message?code=blahmessageid",
+                              expected_url_base.c_str());
+  EXPECT_EQ(expected_url, url.spec());
 }
 
 }  // namespace

chrome/service/cloud_print/cloud_print_proxy.cc

@@ -106,7 +106,7 @@ void CloudPrintProxy::EnableForUser(const std::string& lsid) {
                                   cloud_print_email_, proxy_id);
   }
   if (client_) {
-    client_->OnCloudPrintProxyEnabled();
+    client_->OnCloudPrintProxyEnabled(true);
   }
 }
 
@@ -115,7 +115,7 @@ void CloudPrintProxy::DisableForUser() {
   cloud_print_email_.clear();
   Shutdown();
   if (client_) {
-    client_->OnCloudPrintProxyDisabled();
+    client_->OnCloudPrintProxyDisabled(true);
   }
 }
 
@@ -161,6 +161,15 @@ void CloudPrintProxy::OnAuthenticationFailed() {
       FROM_HERE, NewRunnableFunction(&ShowTokenExpiredNotificationInBrowser));
 }
 
+void CloudPrintProxy::OnPrintSystemUnavailable() {
+  // If the print system is unavailable, we want to shutdown the proxy and
+  // disable it non-persistently.
+  Shutdown();
+  if (client_) {
+    client_->OnCloudPrintProxyDisabled(false);
+  }
+}
+
 void CloudPrintProxy::Shutdown() {
   DCHECK(CalledOnValidThread());
   if (backend_.get())

chrome/service/cloud_print/cloud_print_proxy.h

@@ -23,8 +23,8 @@ class CloudPrintProxy : public CloudPrintProxyFrontend,
   class Client {
    public:
     virtual ~Client() {}
-    virtual void OnCloudPrintProxyEnabled() {}
-    virtual void OnCloudPrintProxyDisabled() {}
+    virtual void OnCloudPrintProxyEnabled(bool persist_state) {}
+    virtual void OnCloudPrintProxyDisabled(bool persist_state) {}
   };
   CloudPrintProxy();
   virtual ~CloudPrintProxy();
@@ -44,13 +44,14 @@ class CloudPrintProxy : public CloudPrintProxyFrontend,
     return cloud_print_email_;
   }
 
-  // Notification methods from the backend. Called on UI thread.
+  // CloudPrintProxyFrontend implementation. Called on UI thread.
   virtual void OnPrinterListAvailable(
       const printing::PrinterList& printer_list);
   virtual void OnAuthenticated(const std::string& cloud_print_token,
                                const std::string& cloud_print_xmpp_token,
                                const std::string& email);
   virtual void OnAuthenticationFailed();
+  virtual void OnPrintSystemUnavailable();
 
  protected:
   void Shutdown();

chrome/service/cloud_print/cloud_print_proxy_backend.h

@@ -34,6 +34,8 @@ class CloudPrintProxyFrontend {
                                const std::string& email) = 0;
   // We have invalid/expired credentials.
   virtual void OnAuthenticationFailed() = 0;
+  // The print system could not be initialized.
+  virtual void OnPrintSystemUnavailable() = 0;
 
  protected:
   // Don't delete through SyncFrontend interface.

chrome/service/cloud_print/print_system.h

@@ -131,6 +131,20 @@ class PrintSystem : public base::RefCountedThreadSafe<PrintSystem> {
                        JobSpooler::Delegate* delegate) = 0;
   };
 
+  class PrintSystemResult {
+   public:
+    PrintSystemResult(bool succeeded, const std::string& message)
+        : succeeded_(succeeded), message_(message) { }
+    bool succeeded() const { return succeeded_; }
+    std::string message() const { return message_; }
+
+   private:
+    bool succeeded_;
+    std::string message_;
+
+    PrintSystemResult() { }
+  };
+
   typedef Callback3<
       bool,
       const std::string&,
@@ -141,7 +155,7 @@ class PrintSystem : public base::RefCountedThreadSafe<PrintSystem> {
 
   // Initialize print system. This need to be called before any other function
   // of PrintSystem.
-  virtual void Init() = 0;
+  virtual PrintSystemResult Init() = 0;
 
   // Enumerates the list of installed local and network printers.
   virtual void EnumeratePrinters(printing::PrinterList* printer_list) = 0;

chrome/service/cloud_print/print_system_cups.cc

@@ -72,7 +72,7 @@ class PrintSystemCUPS : public PrintSystem {
   explicit PrintSystemCUPS(const DictionaryValue* print_system_settings);
 
   // PrintSystem implementation.
-  virtual void Init();
+  virtual PrintSystemResult Init();
 
   virtual void EnumeratePrinters(printing::PrinterList* printer_list);
 
@@ -405,9 +405,10 @@ void PrintSystemCUPS::AddPrintServer(const std::string& url) {
   print_servers_.push_back(print_server);
 }
 
-void PrintSystemCUPS::Init() {
+PrintSystem::PrintSystemResult PrintSystemCUPS::Init() {
   UpdatePrinters();
   initialized_ = true;
+  return PrintSystemResult(true, std::string());
 }
 
 void PrintSystemCUPS::UpdatePrinters() {

chrome/service/cloud_print/print_system_win.cc

@@ -7,6 +7,7 @@
 #include <objidl.h>
 #include <winspool.h>
 
+#include "app/l10n_util.h"
 #include "base/file_path.h"
 #include "base/scoped_ptr.h"
 #include "base/utf_string_conversions.h"
@@ -18,6 +19,7 @@
 #include "chrome/service/service_process.h"
 #include "chrome/service/service_utility_process_host.h"
 #include "gfx/rect.h"
+#include "grit/generated_resources.h"
 #include "printing/backend/print_backend.h"
 #include "printing/backend/print_backend_consts.h"
 #include "printing/backend/win_helper.h"
@@ -246,7 +248,7 @@ class PrintSystemWin : public PrintSystem {
   PrintSystemWin();
 
   // PrintSystem implementation.
-  virtual void Init();
+  virtual PrintSystemResult Init();
 
   virtual void EnumeratePrinters(printing::PrinterList* printer_list);
 
@@ -594,7 +596,13 @@ PrintSystemWin::PrintSystemWin() {
   print_backend_ = printing::PrintBackend::CreateInstance(NULL);
 }
 
-void PrintSystemWin::Init() {
+PrintSystem::PrintSystemResult PrintSystemWin::Init() {
+  if (!printing::XPSModule::Init()) {
+    std::string message = l10n_util::GetStringUTF8(
+        IDS_CLOUD_PRINT_XPS_UNAVAILABLE);
+    return PrintSystemResult(false, message);
+  }
+  return PrintSystemResult(true, std::string());
 }
 
 void PrintSystemWin::EnumeratePrinters(printing::PrinterList* printer_list) {

chrome/service/service_process.cc

@@ -6,6 +6,8 @@
 
 #include <algorithm>
 
+#include "app/app_switches.h"
+#include "app/resource_bundle.h"
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/path_service.h"
@@ -36,6 +38,8 @@ namespace {
 // a shutdown.
 const int64 kShutdownDelay = 60000;
 
+const char kDefaultServiceProcessLocale[] = "en-US";
+
 class ServiceIOThread : public base::Thread {
  public:
   explicit ServiceIOThread(const char* name);
@@ -97,6 +101,21 @@ bool ServiceProcess::Initialize(MessageLoop* message_loop,
       new ServiceProcessPrefs(pref_path, file_thread_->message_loop_proxy()));
   service_prefs_->ReadPrefs();
 
+  // Check if a locale override has been specified on the command-line.
+  std::string locale = command_line.GetSwitchValueASCII(switches::kLang);
+  if (!locale.empty()) {
+    service_prefs_->SetString(prefs::kApplicationLocale, locale);
+    service_prefs_->WritePrefs();
+  } else {
+    // If no command-line value was specified, read the last used locale from
+    // the prefs.
+    service_prefs_->GetString(prefs::kApplicationLocale, &locale);
+    // If no locale was specified anywhere, use the default one.
+    if (locale.empty())
+      locale = kDefaultServiceProcessLocale;
+  }
+  ResourceBundle::InitSharedInstance(locale);
+
 #if defined(ENABLE_REMOTING)
   // Initialize chromoting host manager.
   remoting_host_manager_ = new remoting::ChromotingHostManager(this);
@@ -176,17 +195,21 @@ CloudPrintProxy* ServiceProcess::GetCloudPrintProxy() {
   return cloud_print_proxy_.get();
 }
 
-void ServiceProcess::OnCloudPrintProxyEnabled() {
-  // Save the preference that we have enabled the cloud print proxy.
-  service_prefs_->SetBoolean(prefs::kCloudPrintProxyEnabled, true);
-  service_prefs_->WritePrefs();
+void ServiceProcess::OnCloudPrintProxyEnabled(bool persist_state) {
+  if (persist_state) {
+    // Save the preference that we have enabled the cloud print proxy.
+    service_prefs_->SetBoolean(prefs::kCloudPrintProxyEnabled, true);
+    service_prefs_->WritePrefs();
+  }
   OnServiceEnabled();
 }
 
-void ServiceProcess::OnCloudPrintProxyDisabled() {
-  // Save the preference that we have disabled the cloud print proxy.
-  service_prefs_->SetBoolean(prefs::kCloudPrintProxyEnabled, false);
-  service_prefs_->WritePrefs();
+void ServiceProcess::OnCloudPrintProxyDisabled(bool persist_state) {
+  if (persist_state) {
+    // Save the preference that we have disabled the cloud print proxy.
+    service_prefs_->SetBoolean(prefs::kCloudPrintProxyEnabled, false);
+    service_prefs_->WritePrefs();
+  }
   OnServiceDisabled();
 }
 

chrome/service/service_process.h

@@ -82,8 +82,8 @@ class ServiceProcess : public CloudPrintProxy::Client,
   CloudPrintProxy* GetCloudPrintProxy();
 
   // CloudPrintProxy::Client implementation.
-  virtual void OnCloudPrintProxyEnabled();
-  virtual void OnCloudPrintProxyDisabled();
+  virtual void OnCloudPrintProxyEnabled(bool persist_state);
+  virtual void OnCloudPrintProxyDisabled(bool persist_state);
 
   // ChromotingHostManager::Observer interface.
   virtual void OnRemotingHostEnabled();