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

Unified Diff: third_party/libaddressinput/chromium/cpp/src/country_rules_aggregator.h

Issue 140823005: [rac] Download country code data in a single HTTP request. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add comment. 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 side-by-side diff with in-line comments
Download patch
Index: third_party/libaddressinput/chromium/cpp/src/country_rules_aggregator.h
diff --git a/third_party/libaddressinput/chromium/cpp/src/country_rules_aggregator.h b/third_party/libaddressinput/chromium/cpp/src/country_rules_aggregator.h
index ac4711df802584e23a4e11797b19cc545a32370e..897e8b56d7c537f8c0fb8075a7e0318daa8822de 100644
--- a/third_party/libaddressinput/chromium/cpp/src/country_rules_aggregator.h
+++ b/third_party/libaddressinput/chromium/cpp/src/country_rules_aggregator.h
@@ -19,9 +19,7 @@
#include <libaddressinput/util/basictypes.h>
#include <libaddressinput/util/scoped_ptr.h>
-#include <map>
#include <string>
-#include <vector>
namespace i18n {
namespace addressinput {
@@ -68,8 +66,6 @@ class CountryRulesAggregator {
scoped_ptr<Callback> rules_ready);
private:
- struct RequestData;
-
// Callback for Retriever::Retrieve() method.
void OnDataReady(bool success,
const std::string& key,
@@ -82,35 +78,16 @@ class CountryRulesAggregator {
// a time.
scoped_ptr<Retriever> retriever_;
- // A mapping of data keys (e.g., "data/CA/AB--fr") to information that helps
- // to parse the response data and place it the correct location in the
- // rulesets.
- std::map<std::string, RequestData> requests_;
-
// The country code for which to retrieve the ruleset. Passed to the callback
// method to identify the ruleset. Examples: "US", "CA", "CH", etc.
std::string country_code_;
+ // The key requested from retriever. For example, "data/US".
+ std::string key_;
+
// The callback to invoke when the ruleset has been retrieved.
scoped_ptr<Callback> rules_ready_;
- // The top-level ruleset for the country code. Passed to the callback method
- // as the result of the query.
- scoped_ptr<Ruleset> root_;
-
- // The default language for the country code. This value is parsed from the
- // country-level rule for the country code and is used to filter out the
- // default language from the list of all supported languages for a country.
- // For example, the list of supported languages for Canada is ["en", "fr"],
- // but the default language is "en". Data requests for "data/CA/AB--fr" will
- // succeed, but "data/CA/AB--en" will not return data.
- std::string default_language_;
-
- // The list of all supported languages for the country code. This value is
- // parsed from the country-level rule for the country and is used to download
- // language-specific rules.
- std::vector<std::string> languages_;
-
DISALLOW_COPY_AND_ASSIGN(CountryRulesAggregator);
};

Powered by Google App Engine
This is Rietveld 408576698