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

Side by Side Diff: content/public/browser/access_token_store.h

Issue 8997008: Move AccessTokenStore to Content API, now that it is a pure-virtual class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to head (pure merge). Created 9 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 | Annotate | Revision Log
« no previous file with comments | « content/content_browser.gypi ('k') | content/public/browser/access_token_store.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 // Defines the Geolocation access token store, and associated factory function. 5 // Defines the Geolocation access token store, and associated factory function.
6 // An access token store is responsible for providing the API to persist 6 // An access token store is responsible for providing the API to persist
7 // access tokens, one at a time, and to load them back on mass. 7 // access tokens, one at a time, and to load them back on mass.
8 // The API is a little more complex than one might wish, due to the need for 8 // The API is a little more complex than one might wish, due to the need for
9 // prefs access to happen asynchronously on the UI thread. 9 // prefs access to happen asynchronously on the UI thread.
10 // This API is provided as abstract base classes to allow mocking and testing 10 // This API is provided as abstract base classes to allow mocking and testing
11 // of clients, without dependency on browser process singleton objects etc. 11 // of clients, without dependency on browser process singleton objects etc.
12 12
13 #ifndef CONTENT_BROWSER_GEOLOCATION_ACCESS_TOKEN_STORE_H_ 13 #ifndef CONTENT_PUBLIC_BROWSER_ACCESS_TOKEN_STORE_H_
14 #define CONTENT_BROWSER_GEOLOCATION_ACCESS_TOKEN_STORE_H_ 14 #define CONTENT_PUBLIC_BROWSER_ACCESS_TOKEN_STORE_H_
15 #pragma once 15 #pragma once
16 16
17 #include <map> 17 #include <map>
18 18
19 #include "base/callback.h" 19 #include "base/callback.h"
20 #include "base/memory/ref_counted.h" 20 #include "base/memory/ref_counted.h"
21 #include "base/string16.h" 21 #include "base/string16.h"
22 #include "content/common/content_export.h" 22 #include "content/common/content_export.h"
23 #include "googleurl/src/gurl.h" 23 #include "googleurl/src/gurl.h"
24 24
25 class GURL; 25 class GURL;
26 26
27 namespace net { 27 namespace net {
28 class URLRequestContextGetter; 28 class URLRequestContextGetter;
29 } 29 }
30 30
31 namespace content {
32
31 // Provides storage for the access token used in the network request. 33 // Provides storage for the access token used in the network request.
32 class AccessTokenStore : public base::RefCountedThreadSafe<AccessTokenStore> { 34 class AccessTokenStore : public base::RefCountedThreadSafe<AccessTokenStore> {
33 public: 35 public:
34 // Map of server URLs to associated access token. 36 // Map of server URLs to associated access token.
35 typedef std::map<GURL, string16> AccessTokenSet; 37 typedef std::map<GURL, string16> AccessTokenSet;
36 typedef base::Callback<void(AccessTokenSet, net::URLRequestContextGetter*)> 38 typedef base::Callback<void(AccessTokenSet, net::URLRequestContextGetter*)>
37 LoadAccessTokensCallbackType; 39 LoadAccessTokensCallbackType;
38 40
39 // |callback| will be invoked once per LoadAccessTokens call, after existing 41 // |callback| will be invoked once per LoadAccessTokens call, after existing
40 // access tokens have been loaded from persistent store. As a convenience the 42 // access tokens have been loaded from persistent store. As a convenience the
41 // URLRequestContextGetter is also supplied as an argument in |callback|, as 43 // URLRequestContextGetter is also supplied as an argument in |callback|, as
42 // in Chrome the call to obtain this must also be performed on the UI thread 44 // in Chrome the call to obtain this must also be performed on the UI thread
43 // so it is efficient to piggyback it onto this request. 45 // so it is efficient to piggyback it onto this request.
44 // Takes ownership of |callback|. 46 // Takes ownership of |callback|.
45 CONTENT_EXPORT virtual void LoadAccessTokens( 47 CONTENT_EXPORT virtual void LoadAccessTokens(
46 const LoadAccessTokensCallbackType& callback) = 0; 48 const LoadAccessTokensCallbackType& callback) = 0;
47 49
48 virtual void SaveAccessToken( 50 virtual void SaveAccessToken(
49 const GURL& server_url, const string16& access_token) = 0; 51 const GURL& server_url, const string16& access_token) = 0;
50 52
51 protected: 53 protected:
52 friend class base::RefCountedThreadSafe<AccessTokenStore>; 54 friend class base::RefCountedThreadSafe<AccessTokenStore>;
53 CONTENT_EXPORT AccessTokenStore(); 55 CONTENT_EXPORT AccessTokenStore();
54 CONTENT_EXPORT virtual ~AccessTokenStore(); 56 CONTENT_EXPORT virtual ~AccessTokenStore();
55 57
56 private: 58 private:
57 DISALLOW_COPY_AND_ASSIGN(AccessTokenStore); 59 DISALLOW_COPY_AND_ASSIGN(AccessTokenStore);
58 }; 60 };
59 61
60 #endif // CONTENT_BROWSER_GEOLOCATION_ACCESS_TOKEN_STORE_H_ 62 } // namespace content
63
64 #endif // CONTENT_PUBLIC_BROWSER_ACCESS_TOKEN_STORE_H_
OLDNEW
« no previous file with comments | « content/content_browser.gypi ('k') | content/public/browser/access_token_store.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698