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

Unified Diff: chrome/browser/ui/webui/settings/safe_browsing_handler.cc

Issue 2666093002: Remove base::FundamentalValue (Closed)
Patch Set: Rebase Created 3 years, 10 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: chrome/browser/ui/webui/settings/safe_browsing_handler.cc
diff --git a/chrome/browser/ui/webui/settings/safe_browsing_handler.cc b/chrome/browser/ui/webui/settings/safe_browsing_handler.cc
index fb2e3f9a7e767262a196edeca1bad96879d5e817..918dcf9122146735043bd93c1eee3975527b27f9 100644
--- a/chrome/browser/ui/webui/settings/safe_browsing_handler.cc
+++ b/chrome/browser/ui/webui/settings/safe_browsing_handler.cc
@@ -47,8 +47,7 @@ void SafeBrowsingHandler::HandleGetSafeBrowsingExtendedReporting(
AllowJavascript();
const base::Value* callback_id;
CHECK(args->Get(0, &callback_id));
- base::FundamentalValue is_enabled(
- safe_browsing::IsExtendedReportingEnabled(*prefs_));
+ base::Value is_enabled(safe_browsing::IsExtendedReportingEnabled(*prefs_));
ResolveJavascriptCallback(*callback_id, is_enabled);
}
@@ -64,8 +63,7 @@ void SafeBrowsingHandler::OnPrefChanged(const std::string& pref_name) {
DCHECK(pref_name == prefs::kSafeBrowsingExtendedReportingEnabled ||
pref_name == prefs::kSafeBrowsingScoutReportingEnabled);
- base::FundamentalValue is_enabled(
- safe_browsing::IsExtendedReportingEnabled(*prefs_));
+ base::Value is_enabled(safe_browsing::IsExtendedReportingEnabled(*prefs_));
CallJavascriptFunction(
"cr.webUIListenerCallback",
base::StringValue("safe-browsing-extended-reporting-change"), is_enabled);

Powered by Google App Engine
This is Rietveld 408576698