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

Unified Diff: chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_controller.h

Issue 2971183002: Enforces order for settings reset (Closed)
Patch Set: Add browser test for dependency Created 3 years, 5 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/safe_browsing/settings_reset_prompt/settings_reset_prompt_controller.h
diff --git a/chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_controller.h b/chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_controller.h
index a277bd03c7b0e82ce4a28ddea82f7d4e9929d57d..9496c1dacaa07f6066c3610141228575aafe9cda 100644
--- a/chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_controller.h
+++ b/chrome/browser/safe_browsing/settings_reset_prompt/settings_reset_prompt_controller.h
@@ -83,6 +83,23 @@ class SettingsResetPromptController {
// feature parameters.
void MaybeShowSettingsResetPromptWithDelay();
+// Delegate for MaybeShowSettingsResetPromptWithDelay() that can be overriden
+// by tests that only want to check if the flow for the settings reset prompt
+// will be initiated.
+class SettingsResetPromptDelegate {
+ public:
+ SettingsResetPromptDelegate();
+ virtual ~SettingsResetPromptDelegate();
+
+ virtual void ShowSettingsResetPromptWithDelay() const = 0;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(SettingsResetPromptDelegate);
+};
+
+// Sets the global SettingsResetPromptDelegate, usually for testing.
+void SetSettingsResetPromptDelegate(SettingsResetPromptDelegate* delegate);
+
} // namespace safe_browsing
#endif // CHROME_BROWSER_SAFE_BROWSING_SETTINGS_RESET_PROMPT_SETTINGS_RESET_PROMPT_CONTROLLER_H_

Powered by Google App Engine
This is Rietveld 408576698