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

Unified Diff: components/app_modal/javascript_dialog_manager.h

Issue 2412843002: Simplify the JavaScriptDialogManager. (Closed)
Patch Set: braces 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/app_modal/javascript_dialog_manager.h
diff --git a/components/app_modal/javascript_dialog_manager.h b/components/app_modal/javascript_dialog_manager.h
index 626057da6e96f3b8a7f5423e7870fa0ab4e7be99..9e9dfeb63b7df1f85d134892d9bbe8349182ad19 100644
--- a/components/app_modal/javascript_dialog_manager.h
+++ b/components/app_modal/javascript_dialog_manager.h
@@ -57,9 +57,9 @@ class JavaScriptDialogManager : public content::JavaScriptDialogManager {
bool HandleJavaScriptDialog(content::WebContents* web_contents,
bool accept,
const base::string16* prompt_override) override;
- void CancelActiveAndPendingDialogs(
- content::WebContents* web_contents) override;
- void ResetDialogState(content::WebContents* web_contents) override;
+ void CancelDialogs(content::WebContents* web_contents,
+ bool suppress_callbacks,
+ bool reset_state) override;
base::string16 GetTitle(content::WebContents* web_contents,
const GURL& origin_url,
« no previous file with comments | « components/app_modal/javascript_app_modal_dialog.cc ('k') | components/app_modal/javascript_dialog_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698