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

Unified Diff: chrome/browser/ui/webui/options/automatic_settings_reset_handler.cc

Issue 151003004: Add an automatic settings reset banner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Gab's feedback. Created 6 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/options/automatic_settings_reset_handler.cc
diff --git a/chrome/browser/ui/webui/options/automatic_settings_reset_handler.cc b/chrome/browser/ui/webui/options/automatic_settings_reset_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d0ba27e1387c3c08160c6ab7f91b204593b4f691
--- /dev/null
+++ b/chrome/browser/ui/webui/options/automatic_settings_reset_handler.cc
@@ -0,0 +1,70 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/webui/options/automatic_settings_reset_handler.h"
+
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/prefs/pref_service.h"
+#include "base/time/time.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/common/url_constants.h"
+#include "content/public/browser/web_ui.h"
+#include "grit/generated_resources.h"
+
+namespace {
gab 2014/02/07 22:00:19 Empty line at beginning/end of namespace.
robertshield 2014/02/10 02:30:53 Done.
+const int kBannerShowTimeInDays = 5;
+}
+
+namespace {
gab 2014/02/07 22:00:19 You have two anonymous namespaces :) -- merge this
robertshield 2014/02/10 02:30:53 ugh, tired. thanks, done :)
+
+// Called when the user dismisses the automatic settings reset banner.
+void OnDismissedAutomaticSettingsResetBanner(const base::ListValue* value) {
+ PrefService* local_state = g_browser_process->local_state();
+ local_state->ClearPref(prefs::kProfilePreferenceResetTime);
+}
+
+} // namespace
+
+namespace options {
+
+AutomaticSettingsResetHandler::AutomaticSettingsResetHandler() {}
+AutomaticSettingsResetHandler::~AutomaticSettingsResetHandler() {}
+
+void AutomaticSettingsResetHandler::InitializePage() {
+ const int64 reset_time =
+ g_browser_process->local_state()->GetInt64(
+ prefs::kProfilePreferenceResetTime);
+ if (reset_time != 0) {
+ const base::Time now = base::Time::Now();
+ const base::Time then = base::Time::FromInternalValue(reset_time);
+ if ((now - then).InDays() < kBannerShowTimeInDays)
+ web_ui()->CallJavascriptFunction("AutomaticSettingsResetBanner.show");
+ }
+}
+
+void AutomaticSettingsResetHandler::GetLocalizedValues(
+ base::DictionaryValue* localized_strings) {
+ DCHECK(localized_strings);
+
+ static const OptionsStringResource resources[] = {
+ { "automaticSettingsResetBannerText",
+ IDS_AUTOMATIC_SETTINGS_RESET_BANNER_TEXT },
+ { "automaticSettingsResetLearnMoreUrl",
+ IDS_LEARN_MORE },
+ };
+
+ RegisterStrings(localized_strings, resources, arraysize(resources));
+ localized_strings->SetString(
+ "automaticSettingsResetLearnMoreUrl",
+ chrome::kAutomaticSettingsResetLearnMoreURL);
+}
+
+void AutomaticSettingsResetHandler::RegisterMessages() {
+ web_ui()->RegisterMessageCallback("onDismissedAutomaticSettingsResetBanner",
+ base::Bind(&OnDismissedAutomaticSettingsResetBanner));
+}
+
+} // namespace options

Powered by Google App Engine
This is Rietveld 408576698