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

Side by Side Diff: Source/modules/push_messaging/PushManager.cpp

Issue 687933002: Move push client to frame. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase. Created 6 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "config.h" 5 #include "config.h"
6 #include "modules/push_messaging/PushManager.h" 6 #include "modules/push_messaging/PushManager.h"
7 7
8 #include "bindings/core/v8/CallbackPromiseAdapter.h" 8 #include "bindings/core/v8/CallbackPromiseAdapter.h"
9 #include "bindings/core/v8/ScriptPromise.h" 9 #include "bindings/core/v8/ScriptPromise.h"
10 #include "bindings/core/v8/ScriptPromiseResolver.h" 10 #include "bindings/core/v8/ScriptPromiseResolver.h"
(...skipping 20 matching lines...) Expand all
31 { 31 {
32 } 32 }
33 33
34 // FIXME: This call should be available from workers which will not have a Docum ent object available. 34 // FIXME: This call should be available from workers which will not have a Docum ent object available.
35 // See crbug.com/389194 35 // See crbug.com/389194
36 ScriptPromise PushManager::registerPushMessaging(ScriptState* scriptState) 36 ScriptPromise PushManager::registerPushMessaging(ScriptState* scriptState)
37 { 37 {
38 ASSERT(scriptState->executionContext()->isDocument()); 38 ASSERT(scriptState->executionContext()->isDocument());
39 39
40 Document* document = toDocument(scriptState->executionContext()); 40 Document* document = toDocument(scriptState->executionContext());
41 if (!document->domWindow() || !document->page()) 41 if (!document->domWindow() || !document->frame())
42 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(AbortError, "Document is detached from window.")); 42 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(AbortError, "Document is detached from window."));
43 43
44 WebServiceWorkerProvider* serviceWorkerProvider = NavigatorServiceWorker::se rviceWorker(*document->domWindow()->navigator())->provider(); 44 WebServiceWorkerProvider* serviceWorkerProvider = NavigatorServiceWorker::se rviceWorker(*document->domWindow()->navigator())->provider();
45 if (!serviceWorkerProvider) 45 if (!serviceWorkerProvider)
46 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(AbortError, "No Service Worker installed for this document.")); 46 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(AbortError, "No Service Worker installed for this document."));
47 47
48 // FIXME: Once everything except permission request goes through platform, 48 // FIXME: Once everything except permission request goes through platform,
49 // delete WebPushClient and usage such as this one. 49 // delete WebPushClient and usage such as this one.
50 // See crbug.com/389194 50 // See crbug.com/389194
51 WebPushClient* client = PushController::clientFrom(document->page()); 51 WebPushClient* client = PushController::clientFrom(document->frame());
52 ASSERT(client); 52 ASSERT(client);
53 53
54 RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scrip tState); 54 RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scrip tState);
55 ScriptPromise promise = resolver->promise(); 55 ScriptPromise promise = resolver->promise();
56 56
57 PushPermissionClient* permissionClient = PushPermissionClient::from(scriptSt ate->executionContext()); 57 PushPermissionClient* permissionClient = PushPermissionClient::from(scriptSt ate->executionContext());
58 if (permissionClient) 58 if (permissionClient)
59 permissionClient->requestPermission(scriptState->executionContext(), new PushPermissionRequestCallback(this, client, resolver, serviceWorkerProvider)); 59 permissionClient->requestPermission(scriptState->executionContext(), new PushPermissionRequestCallback(this, client, resolver, serviceWorkerProvider));
60 else 60 else
61 doRegister(client, resolver, serviceWorkerProvider); 61 doRegister(client, resolver, serviceWorkerProvider);
62 62
63 return promise; 63 return promise;
64 } 64 }
65 65
66 // FIXME: This call should be available from workers which will not have a Docum ent object available. 66 // FIXME: This call should be available from workers which will not have a Docum ent object available.
67 // See crbug.com/389194 67 // See crbug.com/389194
68 ScriptPromise PushManager::hasPermission(ScriptState* scriptState) 68 ScriptPromise PushManager::hasPermission(ScriptState* scriptState)
69 { 69 {
70 ASSERT(scriptState->executionContext()->isDocument()); 70 ASSERT(scriptState->executionContext()->isDocument());
71 71
72 Document* document = toDocument(scriptState->executionContext()); 72 Document* document = toDocument(scriptState->executionContext());
73 if (!document->domWindow() || !document->page()) 73 if (!document->domWindow() || !document->frame())
74 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(InvalidStateError, "Document is detached from window.")); 74 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(InvalidStateError, "Document is detached from window."));
75 blink::WebPushClient* client = PushController::clientFrom(document->page()); 75 blink::WebPushClient* client = PushController::clientFrom(document->frame()) ;
76 ASSERT(client); 76 ASSERT(client);
77 77
78 // The currently implemented specification does not require a Service Worker to be present for the 78 // The currently implemented specification does not require a Service Worker to be present for the
79 // hasPermission() call to work, but it will become a requirement soon. 79 // hasPermission() call to work, but it will become a requirement soon.
80 WebServiceWorkerProvider* serviceWorkerProvider = NavigatorServiceWorker::se rviceWorker(*document->domWindow()->navigator())->provider(); 80 WebServiceWorkerProvider* serviceWorkerProvider = NavigatorServiceWorker::se rviceWorker(*document->domWindow()->navigator())->provider();
81 if (!serviceWorkerProvider) 81 if (!serviceWorkerProvider)
82 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(InvalidStateError, "No Service Worker installed for this document.")); 82 return ScriptPromise::rejectWithDOMException(scriptState, DOMException:: create(InvalidStateError, "No Service Worker installed for this document."));
83 83
84 RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scrip tState); 84 RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scrip tState);
85 85
86 ScriptPromise promise = resolver->promise(); 86 ScriptPromise promise = resolver->promise();
87 client->getPermissionStatus(new PushPermissionStatusCallback(resolver), serv iceWorkerProvider); 87 client->getPermissionStatus(new PushPermissionStatusCallback(resolver), serv iceWorkerProvider);
88 return promise; 88 return promise;
89 } 89 }
90 90
91 void PushManager::doRegister(WebPushClient* client, PassRefPtr<ScriptPromiseReso lver> resolver, WebServiceWorkerProvider* serviceWorkerProvider) 91 void PushManager::doRegister(WebPushClient* client, PassRefPtr<ScriptPromiseReso lver> resolver, WebServiceWorkerProvider* serviceWorkerProvider)
92 { 92 {
93 client->registerPushMessaging(new CallbackPromiseAdapter<PushRegistration, P ushError>(resolver), serviceWorkerProvider); 93 client->registerPushMessaging(new CallbackPromiseAdapter<PushRegistration, P ushError>(resolver), serviceWorkerProvider);
94 } 94 }
95 95
96 } // namespace blink 96 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698