Move common cloud print methods from service/cloud_print to common/cloud_print.

chrome/service/cloud_print/cloud_print_connector.cc

Index: chrome/service/cloud_print/cloud_print_connector.cc
diff --git a/chrome/service/cloud_print/cloud_print_connector.cc b/chrome/service/cloud_print/cloud_print_connector.cc
index fa927f34019f35e03c3111b8e6943ff189eefa8c..e99427a05af25f87f236a4a0d825f9b53705648b 100644
--- a/chrome/service/cloud_print/cloud_print_connector.cc
+++ b/chrome/service/cloud_print/cloud_print_connector.cc
@@ -13,6 +13,7 @@
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
+#include "chrome/common/cloud_print/cloud_print_consts.h"
 #include "chrome/common/cloud_print/cloud_print_helpers.h"
 #include "chrome/service/cloud_print/cloud_print_consts.h"
 #include "chrome/service/cloud_print/cloud_print_helpers.h"
@@ -208,9 +209,9 @@ CloudPrintConnector::HandlePrinterListResponse(
       DictionaryValue* printer_data = NULL;
       if (printer_list->GetDictionary(index, &printer_data)) {
         std::string printer_name;
-        printer_data->GetString(kNameValue, &printer_name);
+        printer_data->GetString(cloud_print::kNameValue, &printer_name);
         std::string printer_id;
-        printer_data->GetString(kIdValue, &printer_id);
+        printer_data->GetString(cloud_print::kIdValue, &printer_id);
         if (settings_.IsPrinterBlacklisted(printer_name)) {
           VLOG(1) << "CP_CONNECTOR: Deleting " << printer_name <<
               " id: " << printer_id << " as blacklisted";
@@ -309,11 +310,11 @@ void CloudPrintConnector::ReportUserMessage(const std::string& message_id,
   // Result of this request will be ignored.
   std::string mime_boundary;
   cloud_print::CreateMimeBoundaryForUpload(&mime_boundary);
-  GURL url = CloudPrintHelpers::GetUrlForUserMessage(settings_.server_url(),
-                                                     message_id);
+  GURL url = cloud_print::GetUrlForUserMessage(settings_.server_url(),
+                                               message_id);
   std::string post_data;
-  cloud_print::AddMultipartValueForUpload(kMessageTextValue, failure_msg,
-      mime_boundary, std::string(), &post_data);
+  cloud_print::AddMultipartValueForUpload(cloud_print::kMessageTextValue,
+      failure_msg, mime_boundary, std::string(), &post_data);
   // Terminate the request body
   post_data.append("--" + mime_boundary + "--\r\n");
   std::string mime_type("multipart/form-data; boundary=");
@@ -340,7 +341,8 @@ void CloudPrintConnector::InitJobHandlerForPrinter(
     DictionaryValue* printer_data) {
   DCHECK(printer_data);
   PrinterJobHandler::PrinterInfoFromCloud printer_info_cloud;
-  printer_data->GetString(kIdValue, &printer_info_cloud.printer_id);
+  printer_data->GetString(cloud_print::kIdValue,
+                          &printer_info_cloud.printer_id);
   DCHECK(!printer_info_cloud.printer_id.empty());
   VLOG(1) << "CP_CONNECTOR: Init job handler"
           << ", printer id: " << printer_info_cloud.printer_id;
@@ -350,19 +352,20 @@ void CloudPrintConnector::InitJobHandlerForPrinter(
     return;  // Nothing to do if we already have a job handler for this printer.
 
   printing::PrinterBasicInfo printer_info;
-  printer_data->GetString(kNameValue, &printer_info.printer_name);
+  printer_data->GetString(cloud_print::kNameValue, &printer_info.printer_name);
   DCHECK(!printer_info.printer_name.empty());
-  printer_data->GetString(kPrinterDescValue,
+  printer_data->GetString(cloud_print::kPrinterDescValue,
                           &printer_info.printer_description);
   // Printer status is a string value which actually contains an integer.
   std::string printer_status;
-  if (printer_data->GetString(kPrinterStatusValue, &printer_status)) {
+  if (printer_data->GetString(
+      cloud_print::kPrinterStatusValue, &printer_status)) {
     base::StringToInt(printer_status, &printer_info.printer_status);
   }
-  printer_data->GetString(kPrinterCapsHashValue,
+  printer_data->GetString(cloud_print::kPrinterCapsHashValue,
       &printer_info_cloud.caps_hash);
   ListValue* tags_list = NULL;
-  if (printer_data->GetList(kTagsValue, &tags_list) && tags_list) {
+  if (printer_data->GetList(cloud_print::kTagsValue, &tags_list) && tags_list) {
     for (size_t index = 0; index < tags_list->GetSize(); index++) {
       std::string tag;
       if (tags_list->GetString(index, &tag) &&
@@ -451,7 +454,7 @@ void CloudPrintConnector::ContinuePendingTaskProcessing() {
 }
 
 void CloudPrintConnector::OnPrintersAvailable() {
-  GURL printer_list_url = CloudPrintHelpers::GetUrlForPrinterList(
+  GURL printer_list_url = cloud_print::GetUrlForPrinterList(
       settings_.server_url(), settings_.proxy_id());
   StartGetRequest(printer_list_url,
                   kCloudPrintRegisterMaxRetryCount,
@@ -488,7 +491,7 @@ void CloudPrintConnector::OnPrinterDelete(const std::string& printer_id) {
   // TODO(gene): We probably should not try indefinitely here. Just once or
   // twice should be enough.
   // Bug: http://code.google.com/p/chromium/issues/detail?id=101850
-  GURL url = CloudPrintHelpers::GetUrlForPrinterDelete(
+  GURL url = cloud_print::GetUrlForPrinterDelete(
       settings_.server_url(), printer_id, "printer_deleted");
   StartGetRequest(url,
                   kCloudPrintAPIMaxRetryCount,
@@ -526,26 +529,26 @@ void CloudPrintConnector::OnReceivePrinterCaps(
   cloud_print::CreateMimeBoundaryForUpload(&mime_boundary);
   std::string post_data;
 
-  cloud_print::AddMultipartValueForUpload(kProxyIdValue, settings_.proxy_id(),
-      mime_boundary, std::string(), &post_data);
-  cloud_print::AddMultipartValueForUpload(kPrinterNameValue, info.printer_name,
-      mime_boundary, std::string(), &post_data);
-  cloud_print::AddMultipartValueForUpload(kPrinterDescValue,
-      info.printer_description, mime_boundary, std::string() , &post_data);
-  cloud_print::AddMultipartValueForUpload(kPrinterStatusValue,
+  cloud_print::AddMultipartValueForUpload(cloud_print::kProxyIdValue,
+      settings_.proxy_id(), mime_boundary, std::string(), &post_data);
+  cloud_print::AddMultipartValueForUpload(cloud_print::kPrinterNameValue,
+      info.printer_name, mime_boundary, std::string(), &post_data);
+  cloud_print::AddMultipartValueForUpload(cloud_print::kPrinterDescValue,
+      info.printer_description, mime_boundary, std::string(), &post_data);
+  cloud_print::AddMultipartValueForUpload(cloud_print::kPrinterStatusValue,
       base::StringPrintf("%d", info.printer_status),
       mime_boundary, std::string(), &post_data);
   post_data += CloudPrintHelpers::GetPostDataForPrinterTags(info,
                                                             mime_boundary);
-  cloud_print::AddMultipartValueForUpload(kPrinterCapsValue,
+  cloud_print::AddMultipartValueForUpload(cloud_print::kPrinterCapsValue,
       caps_and_defaults.printer_capabilities, mime_boundary,
       caps_and_defaults.caps_mime_type, &post_data);
-  cloud_print::AddMultipartValueForUpload(kPrinterDefaultsValue,
+  cloud_print::AddMultipartValueForUpload(cloud_print::kPrinterDefaultsValue,
       caps_and_defaults.printer_defaults, mime_boundary,
       caps_and_defaults.defaults_mime_type, &post_data);
   // Send a hash of the printer capabilities to the server. We will use this
   // later to check if the capabilities have changed
-  cloud_print::AddMultipartValueForUpload(kPrinterCapsHashValue,
+  cloud_print::AddMultipartValueForUpload(cloud_print::kPrinterCapsHashValue,
       base::MD5String(caps_and_defaults.printer_capabilities),
       mime_boundary, std::string(), &post_data);
 
@@ -554,7 +557,7 @@ void CloudPrintConnector::OnReceivePrinterCaps(
   std::string mime_type("multipart/form-data; boundary=");
   mime_type += mime_boundary;
 
-  GURL post_url = CloudPrintHelpers::GetUrlForPrinterRegistration(
+  GURL post_url = cloud_print::GetUrlForPrinterRegistration(
       settings_.server_url());
   StartPostRequest(post_url,
                    kCloudPrintAPIMaxRetryCount,