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

Side by Side Diff: public/platform/modules/push_messaging/WebPushSubscription.h

Issue 1222903007: Introduce the PushSubscription.curve25519dh attribute (1/3) (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 5 months 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
« no previous file with comments | « Source/platform/exported/WebPushSubscription.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 WebPushSubscription_h 5 #ifndef WebPushSubscription_h
6 #define WebPushSubscription_h 6 #define WebPushSubscription_h
7 7
8 #include "public/platform/WebCommon.h" 8 #include "public/platform/WebCommon.h"
9 #include "public/platform/WebString.h" 9 #include "public/platform/WebString.h"
10 #include "public/platform/WebURL.h" 10 #include "public/platform/WebURL.h"
11 #include "public/platform/WebVector.h"
11 12
12 namespace blink { 13 namespace blink {
13 14
14 struct WebPushSubscription { 15 struct WebPushSubscription {
16 WebPushSubscription(const WebURL& endpoint, const WebVector<unsigned char>& curve25519dh)
Mike West 2015/07/10 08:06:55 Might be worth adding a comment that |endpoint| mu
Peter Beverloo 2015/07/13 13:58:14 I've mentioned that it must be unique for each sub
17 : endpoint(endpoint)
18 , curve25519dh(curve25519dh)
19 {
20 }
21
22 // TODO(peter): Remove this constructor when the embedder doesn't use it any more.
15 explicit WebPushSubscription(const WebURL& endpoint) 23 explicit WebPushSubscription(const WebURL& endpoint)
16 : endpoint(endpoint) 24 : endpoint(endpoint)
17 { 25 {
18 } 26 }
19 27
20 // TODO(peter): Remove this constructor when the embedder switched over to t he one above.
21 BLINK_PLATFORM_EXPORT WebPushSubscription(const WebString& endpointWithoutSu bscriptionId, const WebString& subscriptionId);
22
23 WebURL endpoint; 28 WebURL endpoint;
29 WebVector<unsigned char> curve25519dh;
johnme 2015/07/09 11:00:15 Actually, this name is a little cryptic. Presumabl
Peter Beverloo 2015/07/09 14:28:34 I clarified the CLs description.
24 }; 30 };
25 31
26 } // namespace blink 32 } // namespace blink
27 33
28 #endif // WebPushSubscription_h 34 #endif // WebPushSubscription_h
OLDNEW
« no previous file with comments | « Source/platform/exported/WebPushSubscription.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698