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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/ntp_snippets/features.cc

Issue 2520853002: [NTP] Cleanup: add brackets in components/ntp_snippets. [2/2] (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « components/ntp_snippets/content_suggestions_service.cc ('k') | components/ntp_snippets/remote/ntp_snippets_status_service.cc » ('j') | components/ntp_snippets/remote/ntp_snippets_status_service.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/ntp_snippets/features.cc
diff --git a/components/ntp_snippets/features.cc b/components/ntp_snippets/features.cc
index e6fbd13a0284c31d24bf200239715bf6b70d2325..fe007b58d22f8c9249b2ccd4e9cb80ef69f7c975 100644
--- a/components/ntp_snippets/features.cc
+++ b/components/ntp_snippets/features.cc
@@ -67,10 +67,12 @@ bool GetParamAsBool(const base::Feature& feature,
bool default_value) {
std::string value_as_string =
variations::GetVariationParamValueByFeature(feature, param_name);
- if (value_as_string == "true")
+ if (value_as_string == "true") {
return true;
- if (value_as_string == "false")
+ }
+ if (value_as_string == "false") {
return false;
+ }
if (!value_as_string.empty()) {
LOG(WARNING) << "Failed to parse variation param " << param_name
« no previous file with comments | « components/ntp_snippets/content_suggestions_service.cc ('k') | components/ntp_snippets/remote/ntp_snippets_status_service.cc » ('j') | components/ntp_snippets/remote/ntp_snippets_status_service.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698