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

Side by Side Diff: Source/modules/push_messaging/PushRegistration.h

Issue 793913002: Stub implementation for PushRegistration.unregister(). (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@mvan_6
Patch Set: rebase Created 6 years 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
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 #ifndef PushRegistration_h 5 #ifndef PushRegistration_h
6 #define PushRegistration_h 6 #define PushRegistration_h
7 7
8 #include "bindings/core/v8/ScriptPromise.h"
8 #include "bindings/core/v8/ScriptWrappable.h" 9 #include "bindings/core/v8/ScriptWrappable.h"
9 #include "platform/heap/Handle.h" 10 #include "platform/heap/Handle.h"
10 #include "public/platform/WebPushRegistration.h" 11 #include "public/platform/WebPushRegistration.h"
11 #include "wtf/text/WTFString.h" 12 #include "wtf/text/WTFString.h"
12 13
13 namespace blink { 14 namespace blink {
14 15
16 class ServiceWorkerRegistration;
15 class ScriptPromiseResolver; 17 class ScriptPromiseResolver;
18 class ScriptState;
16 19
17 class PushRegistration final : public GarbageCollectedFinalized<PushRegistration >, public ScriptWrappable { 20 class PushRegistration final : public GarbageCollectedFinalized<PushRegistration >, public ScriptWrappable {
18 DEFINE_WRAPPERTYPEINFO(); 21 DEFINE_WRAPPERTYPEINFO();
19 public: 22 public:
20 // For CallbackPromiseAdapter. 23 // For CallbackPromiseAdapter.
21 typedef WebPushRegistration WebType; 24 typedef WebPushRegistration WebType;
Michael van Ouwerkerk 2014/12/12 12:18:04 It seems you don't need this for CallbackPromiseAd
mlamouri (slow - plz ping) 2014/12/15 10:55:07 Done.
22 static PushRegistration* take(ScriptPromiseResolver*, WebType* registrationR aw); 25 static PushRegistration* take(ScriptPromiseResolver*, WebType* pushRegistrat ion, ServiceWorkerRegistration*);
23 static void dispose(WebType* registrationRaw); 26 static void dispose(WebType* registrationRaw);
24 27
25 virtual ~PushRegistration(); 28 virtual ~PushRegistration();
26 29
27 const String& pushEndpoint() const { return m_pushEndpoint; } 30 const String& pushEndpoint() const { return m_pushEndpoint; }
28 const String& pushRegistrationId() const { return m_pushRegistrationId; } 31 const String& pushRegistrationId() const { return m_pushRegistrationId; }
32 ScriptPromise unregister(ScriptState*);
29 33
30 void trace(Visitor*) { } 34 void trace(Visitor*);
31 35
32 private: 36 private:
33 PushRegistration(const String& pushEndpoint, const String& pushRegistrationI d); 37 PushRegistration(const String& pushEndpoint, const String& pushRegistrationI d, ServiceWorkerRegistration*);
34 38
35 String m_pushEndpoint; 39 String m_pushEndpoint;
36 String m_pushRegistrationId; 40 String m_pushRegistrationId;
41 Member<ServiceWorkerRegistration> m_registration;
Michael van Ouwerkerk 2014/12/12 12:18:04 Please rename this to m_serviceWorkerRegistration
mlamouri (slow - plz ping) 2014/12/15 10:55:07 Done.
37 }; 42 };
38 43
39 } // namespace blink 44 } // namespace blink
40 45
41 #endif // PushRegistration_h 46 #endif // PushRegistration_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698