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

Unified Diff: chrome/browser/push_messaging/push_messaging_service_impl.cc

Issue 1931843002: Ship Web Push subscription restrictions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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: chrome/browser/push_messaging/push_messaging_service_impl.cc
diff --git a/chrome/browser/push_messaging/push_messaging_service_impl.cc b/chrome/browser/push_messaging/push_messaging_service_impl.cc
index 988cccadd939412a1583541fd4f12de67c7cd5d2..23a8d14950b464a0b74c440d5a0387d89497eb61 100644
--- a/chrome/browser/push_messaging/push_messaging_service_impl.cc
+++ b/chrome/browser/push_messaging/push_messaging_service_impl.cc
@@ -334,10 +334,11 @@ void PushMessagingServiceImpl::OnSendAcknowledged(
NOTREACHED() << "The Push API shouldn't have sent messages upstream";
}
-// GetPushEndpoint method ------------------------------------------------------
+// GetEndpoint method ----------------------------------------------------------
-GURL PushMessagingServiceImpl::GetPushEndpoint() {
- return GURL(std::string(kPushMessagingEndpoint));
+GURL PushMessagingServiceImpl::GetEndpoint(bool standard_protocol) const {
+ return GURL(standard_protocol ? kPushMessagingPushProtocolEndpoint
+ : kPushMessagingGcmEndpoint);
}
// Subscribe and GetPermissionStatus methods -----------------------------------

Powered by Google App Engine
This is Rietveld 408576698