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

Unified Diff: components/ntp_snippets/remote/ntp_snippets_fetcher.h

Issue 2449873003: [NTP Snippets] Another round of clang-tidy cleanups (Closed)
Patch Set: Created 4 years, 2 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: components/ntp_snippets/remote/ntp_snippets_fetcher.h
diff --git a/components/ntp_snippets/remote/ntp_snippets_fetcher.h b/components/ntp_snippets/remote/ntp_snippets_fetcher.h
index f5bbf27412cc229053c8a01e15988a324e07afc0..da4a23cc3337ad2d6a01086ca0a81cbc155599ce 100644
--- a/components/ntp_snippets/remote/ntp_snippets_fetcher.h
+++ b/components/ntp_snippets/remote/ntp_snippets_fetcher.h
@@ -169,17 +169,18 @@ class NTPSnippetsFetcher : public OAuth2TokenService::Consumer,
};
struct RequestParams {
- FetchAPI fetch_api;
+ FetchAPI fetch_api = CHROME_READER_API;
std::string obfuscated_gaia_id;
- bool only_return_personalized_results;
+ bool only_return_personalized_results = false;
std::string user_locale;
std::set<std::string> host_restricts;
std::set<std::string> excluded_ids;
- int count_to_fetch;
- bool interactive_request;
+ int count_to_fetch = 0;
+ bool interactive_request = false;
std::string user_class;
- translate::LanguageModel::LanguageInfo ui_language;
- translate::LanguageModel::LanguageInfo other_top_language;
+ translate::LanguageModel::LanguageInfo ui_language = {std::string(), 0.0f};
Bernhard Bauer 2016/10/26 08:52:04 Hm. I'd be tempted to instead give LanguageInfo an
Marc Treib 2016/10/26 09:19:18 You're right of course - I was being lazy and not
+ translate::LanguageModel::LanguageInfo other_top_language = {std::string(),
+ 0.0f};
Marc Treib 2016/10/26 08:23:54 All this isn't completely clang-tidy-mandated - it
RequestParams();
~RequestParams();
« no previous file with comments | « components/ntp_snippets/remote/ntp_snippets_database.cc ('k') | components/ntp_snippets/remote/ntp_snippets_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698