Refactor chrome_launcher_support::GetAnyChromePath.

cloud_print/service/win/service_listener.cc

Index: cloud_print/service/win/service_listener.cc
diff --git a/cloud_print/service/win/service_listener.cc b/cloud_print/service/win/service_listener.cc
index 0aaf0f305848d531318f0a293f19b23841129cfc..f80e75982fd66e0c94e42318e7c2b6709d929403 100644
--- a/cloud_print/service/win/service_listener.cc
+++ b/cloud_print/service/win/service_listener.cc
@@ -35,8 +35,9 @@ std::string GetEnvironment(const base::FilePath& user_data_dir) {
                          printing::XPSModule::Init());
   environment.SetString(SetupListener::kUserNameJsonValueName,
                         GetCurrentUserName());
-  environment.SetString(SetupListener::kChromePathJsonValueName,
-                        chrome_launcher_support::GetAnyChromePath().value());
+  environment.SetString(
+      SetupListener::kChromePathJsonValueName,
+      chrome_launcher_support::GetAnyChromePath(false).value());

[grt (UTC plus 2), 2015/01/09 17:45:28]

there was discussion of canary cloud print elsewhere in the review comments.
should this somehow differentiate a canary cloud print service from a
non-canary?

is special handling needed here in case GetAnyChromePath returns an empty path?

[Matt Giuca, 2015/01/12 06:54:26]

Same as the discussion on chrome_launcher.cc: if there are bugs here, they
should be handled separately to this CL, which is purely a refactor.

   if (base::CreateDirectory(user_data_dir)) {
     base::FilePath temp_file;
     if (base::CreateTemporaryFileInDir(user_data_dir, &temp_file)) {