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

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, 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
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 95aac69c1233be8e27ca219244e1b78dc6860d32..cad219da02995ba93c51019f8921c84b6abc69d4 100644
--- a/chrome/browser/push_messaging/push_messaging_service_impl.cc
+++ b/chrome/browser/push_messaging/push_messaging_service_impl.cc
@@ -349,10 +349,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