Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(762)

Unified Diff: content/public/common/push_messaging_status.cc

Issue 340773006: Dispatch push event to worker from PushServiceImpl#OnMessage. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Get registration from storage if necessary. Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 0040072bc0806e7ff883b298b2ca2a6ca8f894f3..4baeaaa7e6f6c19e8dd8e4ef0de4ff5a23d7c1ef 100644
--- a/content/public/common/push_messaging_status.cc
+++ b/content/public/common/push_messaging_status.cc
@@ -28,6 +28,12 @@ const char* PushMessagingStatusToString(PushMessagingStatus status) {
case PUSH_MESSAGING_STATUS_REGISTRATION_FAILED_SERVICE_ERROR:
return "Registration failed - push service error";
+ case PUSH_MESSAGING_STATUS_MESSAGE_DELIVERY_FAILED_NO_SERVICE_WORKER:
+ return "Message delivery failed because no Service Worker was found";
johnme 2014/07/23 14:25:23 Nit: "Message delivery failed - no matching Servic
Michael van Ouwerkerk 2014/07/23 16:57:52 Done.
+
+ case PUSH_MESSAGING_STATUS_MESSAGE_DELIVERY_FAILED_SERVICE_WORKER_ERROR:
+ return "Message delivery failed due to a Service Worker error";
+
case PUSH_MESSAGING_STATUS_ERROR:
return "Operation has failed (unspecified reason)";
}

Powered by Google App Engine
This is Rietveld 408576698