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 892910852f2746d513ffe921f66229fdfe6c13db..af5d353ca5dd451d46d832c0eb8b02074d23df42 100644
--- a/public/platform/WebPushProvider.h
+++ b/public/platform/WebPushProvider.h
@@ -37,14 +37,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