Blink implementation of PushManager#hasPermission()

Source/modules/push_messaging/PushManager.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "modules/push_messaging/PushManager.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "modules/push_messaging/PushController.h"
 #include "modules/push_messaging/PushError.h"
+#include "modules/push_messaging/PushPermissionCallback.h"
 #include "modules/push_messaging/PushRegistration.h"
 #include "modules/serviceworkers/NavigatorServiceWorker.h"
 #include "modules/serviceworkers/ServiceWorkerContainer.h"
 #include "public/platform/WebPushClient.h"
 #include "wtf/RefPtr.h"
 
+
+// Implementation of the Push Api.
+// FIXME: These calls should be available from workers which will not have a Document object available.

[mlamouri (slow - plz ping), 2014/10/17 09:22:01]

I would move that FIXME in the two places where the assumption is made. The file
might evolve in a way that will make this comment outdated.

Also, I don't think it's common to have general comment like "Implementation of
the Push Api."

[Miguel Garcia, 2014/10/20 09:58:10]

Done.

 namespace blink {
 
 PushManager::PushManager()
 {
 }
 
 ScriptPromise PushManager::registerPushMessaging(ScriptState* scriptState, const String& senderId)
 {
     ASSERT(scriptState->executionContext()->isDocument());
 
     Document* document = toDocument(scriptState->executionContext());
     if (!document->domWindow() || !document->page())
         return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(AbortError, "Document is detached from window."));
 
     WebServiceWorkerProvider* serviceWorkerProvider = NavigatorServiceWorker::serviceWorker(document->domWindow()->navigator())->provider();
     if (!serviceWorkerProvider)
         return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(AbortError, "No Service Worker installed for this document."));
 
     WebPushClient* client = PushController::clientFrom(document->page());
     ASSERT(client);
 
     RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
     client->registerPushMessaging(senderId, new CallbackPromiseAdapter<PushRegistration, PushError>(resolver), serviceWorkerProvider);
     return promise;
 }
 
+
+ScriptPromise PushManager::permissionStatus(ScriptState* scriptState)
+{
+    ASSERT(scriptState->executionContext()->isDocument());
+
+    Document* document = toDocument(scriptState->executionContext());
+    if (!document->domWindow() || !document->page())
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(AbortError, "Document is detached from window."));

[mlamouri (slow - plz ping), 2014/10/17 09:22:00]

I think rejecting with "InvalidStateError" would be more appropriate.

[Miguel Garcia, 2014/10/20 09:58:10]

Done.

+    blink::WebPushClient* client = PushController::clientFrom(document->page());
+    ASSERT(client);
+
+    WebServiceWorkerProvider* serviceWorkerProvider = NavigatorServiceWorker::serviceWorker(document->domWindow()->navigator())->provider();

[mlamouri (slow - plz ping), 2014/10/17 09:22:00]

The way to access ServiceWorkerContainer is confusing: you can get the
NavigatorServiceWorker from a Document or a Navigator but you must pass a
Navigator to get the ServiceWorkerContainer. I wonder if it would make sense to
expose the private serviceWorker() as a public method.

+    if (!serviceWorkerProvider)
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(AbortError, "No Service Worker installed for this document."));

[mlamouri (slow - plz ping), 2014/10/17 09:22:01]

ditto: InvalidStateError is probably more appropriate.

[Miguel Garcia, 2014/10/20 09:58:10]

Done.

+
+    RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+
+    client->permissionStatus(new PushPermissionCallback(resolver), serviceWorkerProvider);
+    return resolver->promise();
+}
+
 } // namespace blink