PushRegistration.unregister(), cleanups and more tests.

public/platform/WebPushProvider.h

Index: public/platform/WebPushProvider.h
diff --git a/public/platform/WebPushProvider.h b/public/platform/WebPushProvider.h
index 43f1f78afe1b54e61d7fd716493de5d66667c083..76f28892099b80cce4e8f8c628c80b858578c172 100644
--- a/public/platform/WebPushProvider.h
+++ b/public/platform/WebPushProvider.h
@@ -41,14 +41,7 @@ public:
 
     // Takes ownership if the WebPushUnregisterCallbacks.
     // Does not take ownership of the WebServiceWorkerRegistration.
-    virtual void unregister(WebServiceWorkerRegistration*, WebPushUnregisterCallbacks* callback)
-    {
-        // FIXME: this should call BLINK_ASSERT_NOT_REACHED() when Chromium will
-        // have an implementation.
-        bool result = true;
-        callback->onSuccess(&result);
-        delete callback;
-    }
+    virtual void unregister(WebServiceWorkerRegistration*, WebPushUnregisterCallbacks* callback) { BLINK_ASSERT_NOT_REACHED(); }
 };
 
 } // namespace blink