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

Unified Diff: public/platform/modules/push_messaging/WebPushSubscription.h

Issue 1148763005: Remove support for PushSubscription.subscriptionId. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 7 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
« no previous file with comments | « Source/platform/exported/WebPushSubscription.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: public/platform/modules/push_messaging/WebPushSubscription.h
diff --git a/public/platform/modules/push_messaging/WebPushSubscription.h b/public/platform/modules/push_messaging/WebPushSubscription.h
index 0c4f4efdb788dbc219dce3a146a1ce167e859365..b582bf4c92b11a7dab59a96c287ae03e2ca29569 100644
--- a/public/platform/modules/push_messaging/WebPushSubscription.h
+++ b/public/platform/modules/push_messaging/WebPushSubscription.h
@@ -5,19 +5,22 @@
#ifndef WebPushSubscription_h
#define WebPushSubscription_h
+#include "public/platform/WebCommon.h"
#include "public/platform/WebString.h"
+#include "public/platform/WebURL.h"
namespace blink {
struct WebPushSubscription {
- WebPushSubscription(const WebString& endpoint, const WebString& subscriptionId)
+ explicit WebPushSubscription(const WebURL& endpoint)
: endpoint(endpoint)
- , subscriptionId(subscriptionId)
{
}
- WebString endpoint;
- WebString subscriptionId;
+ // TODO(peter): Remove this constructor when the embedder switched over to the one above.
+ BLINK_PLATFORM_EXPORT WebPushSubscription(const WebString& endpointWithoutSubscriptionId, const WebString& subscriptionId);
+
+ WebURL endpoint;
};
} // namespace blink
« no previous file with comments | « Source/platform/exported/WebPushSubscription.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698