Clean up deprecated Web Push API code-paths.

Remove the Blink API implementations which still accepted a WebString
for the sender Id, which is not being used anymore.

BUG=424601

Review URL: https://codereview.chromium.org/663813002

Cr-Commit-Position: refs/heads/master@{#301348}

chrome/test/data/push_messaging/test.html

@@ -24,8 +24,7 @@
 
     function registerPush() {
       navigator.serviceWorker.ready.then(function() {
-        // TODO(peter): Remove the argument once Blink no longer requires it.
-        navigator.push.register('DEPRECATED').then(function(pushRegistration) {
+        navigator.push.register().then(function(pushRegistration) {
           sendResultToTest(pushRegistration.pushEndpoint + ' - ' + pushRegistration.pushRegistrationId);
         }, sendErrorToTest);
       }, sendErrorToTest);

content/renderer/push_messaging_dispatcher.cc

@@ -37,13 +37,6 @@ bool PushMessagingDispatcher::OnMessageReceived(const IPC::Message& message) {
   return handled;
 }
 
-void PushMessagingDispatcher::registerPushMessaging(
-    const WebString& sender_id,
-    blink::WebPushRegistrationCallbacks* callbacks,
-    blink::WebServiceWorkerProvider* service_worker_provider) {
-  registerPushMessaging(callbacks, service_worker_provider);
-}
-
 void PushMessagingDispatcher::registerPushMessaging(
     blink::WebPushRegistrationCallbacks* callbacks,
     blink::WebServiceWorkerProvider* service_worker_provider) {

content/renderer/push_messaging_dispatcher.h

@@ -20,7 +20,6 @@ class Message;
 
 namespace blink {
 class WebServiceWorkerProvider;
-class WebString;
 }  // namespace blink
 
 namespace content {
@@ -38,11 +37,6 @@ class PushMessagingDispatcher : public RenderFrameObserver,
   bool OnMessageReceived(const IPC::Message& message) override;
 
   // WebPushClient implementation.
-  // TODO(peter): Remove this signature of registerPushMessaging.
-  virtual void registerPushMessaging(
-      const blink::WebString& sender_id,
-      blink::WebPushRegistrationCallbacks* callbacks,
-      blink::WebServiceWorkerProvider* service_worker_provider);
   virtual void registerPushMessaging(
       blink::WebPushRegistrationCallbacks* callbacks,
       blink::WebServiceWorkerProvider* service_worker_provider);

content/shell/renderer/test_runner/mock_web_push_client.cc

@@ -34,13 +34,6 @@ void MockWebPushClient::SetMockErrorValues(const std::string& message) {
   error_message_ = message;
 }
 
-void MockWebPushClient::registerPushMessaging(
-    const WebString& sender_id,
-    blink::WebPushRegistrationCallbacks* callbacks,
-    blink::WebServiceWorkerProvider* service_worker_provider) {
-  registerPushMessaging(callbacks, service_worker_provider);
-}
-
 void MockWebPushClient::registerPushMessaging(
     blink::WebPushRegistrationCallbacks* callbacks,
     blink::WebServiceWorkerProvider* service_worker_provider) {

content/shell/renderer/test_runner/mock_web_push_client.h

@@ -11,7 +11,6 @@
 
 namespace blink {
 class WebServiceWorkerProvider;
-class WebString;
 }  // namespace blink
 
 namespace content {
@@ -35,10 +34,6 @@ class MockWebPushClient : public blink::WebPushClient {
 
  private:
   // WebPushClient implementation.
-  virtual void registerPushMessaging(
-      const blink::WebString& sender_id,
-      blink::WebPushRegistrationCallbacks* callbacks,
-      blink::WebServiceWorkerProvider* service_worker_provider);
   virtual void registerPushMessaging(
       blink::WebPushRegistrationCallbacks* callbacks,
       blink::WebServiceWorkerProvider* service_worker_provider);