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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 793403002: Implement WebPushProvider.unregister() in Chromium. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mvan_2
Patch Set: review commetns Created 6 years 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
« chrome/test/data/push_messaging/push_test.js ('K') | « content/public/browser/push_messaging_service.h ('k') | content/shell/browser/layout_test/layout_test_push_messaging_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/common/push_messaging_status.h
diff --git a/content/public/common/push_messaging_status.h b/content/public/common/push_messaging_status.h
index c55b1efdd86fb0a48f8c392cd9d187fb880ffb65..f4de0e33f2ac984533f63667765e9825366b182b 100644
--- a/content/public/common/push_messaging_status.h
+++ b/content/public/common/push_messaging_status.h
@@ -72,6 +72,21 @@ enum PushDeliveryStatus {
PUSH_DELIVERY_STATUS_LAST = PUSH_DELIVERY_STATUS_EVENT_WAITUNTIL_REJECTED
};
+// Push unregistration success / error codes for internal use & reporting.
+enum PushUnregistrationStatus {
+ // The unregistration was successful.
+ PUSH_UNREGISTRATION_SUCCESS_UNREGISTER,
+
+ // The registration was already unregistered.
+ PUSH_UNREGISTRATION_SUCCESS_WAS_UNREGISTERED,
+
+ // The unregistration did not happen because of a network error.
+ PUSH_UNREGISTRATION_NETWORK_ERROR,
+
+ // The unregistration did not happen because of a miscellaneous error.
+ PUSH_UNREGISTRATION_UNKNOWN_ERROR,
+};
+
const char* PushRegistrationStatusToString(PushRegistrationStatus status);
} // namespace content
« chrome/test/data/push_messaging/push_test.js ('K') | « content/public/browser/push_messaging_service.h ('k') | content/shell/browser/layout_test/layout_test_push_messaging_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698