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

Unified Diff: content/browser/content_browser_client.cc

Issue 7096016: Remove JS dialog dependency from content. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: one more DEPS item removal Created 9 years, 7 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: content/browser/content_browser_client.cc
diff --git a/content/browser/content_browser_client.cc b/content/browser/content_browser_client.cc
index 054173599d7eff3b452a30cc13b55cd6360c2a3f..27ae8731c103f25ecf202ea796fca5c3e95df30e 100644
--- a/content/browser/content_browser_client.cc
+++ b/content/browser/content_browser_client.cc
@@ -5,6 +5,7 @@
#include "content/browser/content_browser_client.h"
#include "base/memory/singleton.h"
+#include "content/browser/javascript_dialog_delegate.h"
#include "content/browser/webui/empty_web_ui_factory.h"
#include "googleurl/src/gurl.h"
@@ -93,4 +94,26 @@ int ContentBrowserClient::GetCrashSignalFD(const std::string& process_type) {
}
#endif
+void ContentBrowserClient::RunJavaScriptDialog(
+ JavaScriptDialogDelegate* delegate,
+ const GURL& frame_url,
+ int dialog_flags,
+ const string16& message_text,
+ const string16& default_prompt_text,
+ IPC::Message* reply_message,
+ bool* did_suppress_message,
+ Profile* profile) {
+ *did_suppress_message = true;
+}
+
+void ContentBrowserClient::RunBeforeUnloadDialog(
+ JavaScriptDialogDelegate* delegate,
+ const string16& message_text,
+ IPC::Message* reply_msg) {
jam 2011/06/01 19:20:08 shouldn't the default implementation call delegate
Avi (use Gerrit) 2011/06/01 19:40:34 Done.
+}
+
+void ContentBrowserClient::ResetJavaScriptState(
+ JavaScriptDialogDelegate* delegate) {
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698