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

Side by Side Diff: third_party/libaddressinput/chromium/cpp/src/retriever.h

Issue 115523011: [rAc - libaddressinput] slay a Helper class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: assert instead Created 6 years, 11 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 | « no previous file | third_party/libaddressinput/chromium/cpp/src/retriever.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) 2013 Google Inc. 1 // Copyright (C) 2013 Google Inc.
2 // 2 //
3 // Licensed under the Apache License, Version 2.0 (the "License"); 3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with the License. 4 // you may not use this file except in compliance with the License.
5 // You may obtain a copy of the License at 5 // You may obtain a copy of the License at
6 // 6 //
7 // http://www.apache.org/licenses/LICENSE-2.0 7 // http://www.apache.org/licenses/LICENSE-2.0
8 // 8 //
9 // Unless required by applicable law or agreed to in writing, software 9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS, 10 // distributed under the License is distributed on an "AS IS" BASIS,
11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12 // See the License for the specific language governing permissions and 12 // See the License for the specific language governing permissions and
13 // limitations under the License. 13 // limitations under the License.
14 // 14 //
15 // An object to retrieve data. 15 // An object to retrieve data.
16 16
17 #ifndef I18N_ADDRESSINPUT_RETRIEVER_H_ 17 #ifndef I18N_ADDRESSINPUT_RETRIEVER_H_
18 #define I18N_ADDRESSINPUT_RETRIEVER_H_ 18 #define I18N_ADDRESSINPUT_RETRIEVER_H_
19 19
20 #include <libaddressinput/callback.h> 20 #include <libaddressinput/callback.h>
21 #include <libaddressinput/util/basictypes.h> 21 #include <libaddressinput/util/basictypes.h>
22 #include <libaddressinput/util/scoped_ptr.h> 22 #include <libaddressinput/util/scoped_ptr.h>
23 23
24 #include <map>
24 #include <string> 25 #include <string>
25 26
26 #include "lookup_key_util.h" 27 #include "lookup_key_util.h"
27 28
28 namespace i18n { 29 namespace i18n {
29 namespace addressinput { 30 namespace addressinput {
30 31
31 class Downloader; 32 class Downloader;
32 class Storage; 33 class Storage;
33 34
34 // Retrieves data. Sample usage: 35 // Retrieves data. Sample usage:
35 // Storage* storage = ...; 36 // Storage* storage = ...;
36 // Downloader* downloader = ...; 37 // Downloader* downloader = ...;
37 // Retriever retriever("https://i18napis.appspot.com/ssl-address/", 38 // Retriever retriever("https://i18napis.appspot.com/ssl-address/",
38 // downloader, storage); 39 // downloader, storage);
39 // retriever.Retrieve("data/CA/AB--fr", 40 // retriever.Retrieve("data/CA/AB--fr",
40 // BuildCallback(this, &MyClass::OnDataRetrieved)); 41 // BuildCallback(this, &MyClass::OnDataRetrieved));
41 class Retriever { 42 class Retriever {
42 public: 43 public:
43 typedef i18n::addressinput::Callback<std::string, std::string> Callback; 44 typedef i18n::addressinput::Callback<std::string, std::string> Callback;
44 45
45 Retriever(const std::string& validation_data_url, 46 Retriever(const std::string& validation_data_url,
46 scoped_ptr<const Downloader> downloader, 47 scoped_ptr<const Downloader> downloader,
47 scoped_ptr<Storage> storage); 48 scoped_ptr<Storage> storage);
48 ~Retriever(); 49 ~Retriever();
49 50
50 // Retrieves the data for |key| and invokes the |retrieved| callback. Checks 51 // Retrieves the data for |key| and invokes the |retrieved| callback. Checks
51 // for the data in storage first. If storage does not have the data for |key|, 52 // for the data in storage first. If storage does not have the data for |key|,
52 // then downloads the data and places it in storage. 53 // then downloads the data and places it in storage.
53 void Retrieve(const std::string& key, scoped_ptr<Callback> retrieved) const; 54 void Retrieve(const std::string& key, scoped_ptr<Callback> retrieved);
54 55
55 private: 56 private:
57 // Callback for when a rule is retrieved from |storage_|.
58 void OnDataRetrievedFromStorage(bool success,
59 const std::string& key,
60 const std::string& data);
61
62 // Callback for when a rule is retrieved by |downloader_|.
63 void OnDownloaded(bool success,
64 const std::string& url,
65 const std::string& data);
66
67 // Looks up the callback for |key| in |requests_|, removes it from the map
68 // and returns it. Assumes that |key| is in fact in |requests_|.
69 scoped_ptr<Callback> GetCallbackForKey(const std::string& key);
70
56 const LookupKeyUtil lookup_key_util_; 71 const LookupKeyUtil lookup_key_util_;
57 scoped_ptr<const Downloader> downloader_; 72 scoped_ptr<const Downloader> downloader_;
58 scoped_ptr<Storage> storage_; 73 scoped_ptr<Storage> storage_;
74 // Holds pending requests. The callback pointers are owned.
75 std::map<std::string, Callback*> requests_;
59 76
60 DISALLOW_COPY_AND_ASSIGN(Retriever); 77 DISALLOW_COPY_AND_ASSIGN(Retriever);
61 }; 78 };
62 79
63 } // namespace addressinput 80 } // namespace addressinput
64 } // namespace i18n 81 } // namespace i18n
65 82
66 #endif // I18N_ADDRESSINPUT_RETRIEVER_H_ 83 #endif // I18N_ADDRESSINPUT_RETRIEVER_H_
OLDNEW
« no previous file with comments | « no previous file | third_party/libaddressinput/chromium/cpp/src/retriever.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698