Split PushMessagingStatus enum according to use case

content/public/common/push_messaging_status.cc

Index: content/public/common/push_messaging_status.cc
diff --git a/content/public/common/push_messaging_status.cc b/content/public/common/push_messaging_status.cc
index f01cf80ca1fe4ef2f49ea17b6f45f25a5615c02a..7c476408d6ea8b6c6949e01051198f8703973f81 100644
--- a/content/public/common/push_messaging_status.cc
+++ b/content/public/common/push_messaging_status.cc
@@ -8,34 +8,25 @@
 
 namespace content {
 
-const char* PushMessagingStatusToString(PushMessagingStatus status) {
+const char* PushRegistrationStatusToString(PushRegistrationStatus status) {
   switch (status) {
-    case PUSH_MESSAGING_STATUS_OK:
-      return "Operation has succeeded";
+    case PUSH_REGISTRATION_STATUS_SUCCESS:
+      return "Registration successful";
 

[palmer, 2014/10/16 23:07:01]

Whitespace nit: Whatever git cl format says

[johnme, 2014/10/17 12:18:25]

git cl format didn't make any changes here (though it found a few places
elsewhere that could be simplified). The blank lines seem to aid readability
slightly?

-    case PUSH_MESSAGING_STATUS_REGISTRATION_FAILED_NO_SERVICE_WORKER:
+    case PUSH_REGISTRATION_STATUS_NO_SERVICE_WORKER:
       return "Registration failed - no Service Worker";
 
-    case PUSH_MESSAGING_STATUS_REGISTRATION_FAILED_SERVICE_NOT_AVAILABLE:
+    case PUSH_REGISTRATION_STATUS_SERVICE_NOT_AVAILABLE:
       return "Registration failed - push service not available";
 
-    case PUSH_MESSAGING_STATUS_REGISTRATION_FAILED_LIMIT_REACHED:
+    case PUSH_REGISTRATION_STATUS_LIMIT_REACHED:
       return "Registration failed - registration limit has been reached";
 
-    case PUSH_MESSAGING_STATUS_REGISTRATION_FAILED_PERMISSION_DENIED:
+    case PUSH_REGISTRATION_STATUS_PERMISSION_DENIED:
       return "Registration failed - permission denied";
 
-    case PUSH_MESSAGING_STATUS_REGISTRATION_FAILED_SERVICE_ERROR:
+    case PUSH_REGISTRATION_STATUS_SERVICE_ERROR:
       return "Registration failed - push service error";
-
-    case PUSH_MESSAGING_STATUS_MESSAGE_DELIVERY_FAILED_NO_SERVICE_WORKER:
-      return "Message delivery failed - no Service Worker";
-
-    case PUSH_MESSAGING_STATUS_MESSAGE_DELIVERY_FAILED_SERVICE_WORKER_ERROR:
-      return "Message delivery failed - Service Worker error";
-
-    case PUSH_MESSAGING_STATUS_ERROR:
-      return "Operation has failed (unspecified reason)";
   }
   NOTREACHED();
   return "";