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

Unified Diff: chrome/browser/app_modal_dialog.cc

Issue 63033: Refactor AppModalDialogQueue and move JS Alert boxes into a MVC. (Closed)
Patch Set: whitespace Created 11 years, 8 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
« no previous file with comments | « chrome/browser/app_modal_dialog.h ('k') | chrome/browser/app_modal_dialog_queue.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/app_modal_dialog.cc
diff --git a/chrome/browser/app_modal_dialog.cc b/chrome/browser/app_modal_dialog.cc
new file mode 100644
index 0000000000000000000000000000000000000000..91cffdd4d91b32a6e6415c6a6b6aec1fa1ea05ff
--- /dev/null
+++ b/chrome/browser/app_modal_dialog.cc
@@ -0,0 +1,99 @@
+// Copyright (c) 2009 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/app_modal_dialog.h"
+
+#include "chrome/browser/app_modal_dialog_queue.h"
+#include "chrome/browser/tab_contents/web_contents.h"
+#include "chrome/common/notification_service.h"
+#include "chrome/common/notification_type.h"
+#include "chrome/common/ipc_message.h"
+
+AppModalDialog::AppModalDialog(WebContents* web_contents,
+ const std::wstring& title,
+ int dialog_flags,
+ const std::wstring& message_text,
+ const std::wstring& default_prompt_text,
+ bool display_suppress_checkbox,
+ bool is_before_unload_dialog,
+ IPC::Message* reply_msg)
+ : web_contents_(web_contents),
+ title_(title),
+ dialog_flags_(dialog_flags),
+ message_text_(message_text),
+ default_prompt_text_(default_prompt_text),
+ display_suppress_checkbox_(display_suppress_checkbox),
+ is_before_unload_dialog_(is_before_unload_dialog),
+ reply_msg_(reply_msg) {
+ InitNotifications();
+}
+
+void AppModalDialog::Observe(NotificationType type,
+ const NotificationSource& source,
+ const NotificationDetails& details) {
+ if (!web_contents_)
+ return;
+
+ if (type == NotificationType::NAV_ENTRY_COMMITTED &&
+ Source<NavigationController>(source).ptr() == web_contents_->controller())
+ web_contents_ = NULL;
+
+ if (type == NotificationType::TAB_CONTENTS_DESTROYED &&
+ Source<TabContents>(source).ptr() ==
+ static_cast<TabContents*>(web_contents_))
+ web_contents_ = NULL;
+
+ if (!web_contents_)
+ CloseModalDialog();
+}
+
+void AppModalDialog::InitNotifications() {
+ // Make sure we get navigation notifications so we know when our parent
+ // contents will disappear or navigate to a different page.
+ registrar_.Add(this, NotificationType::NAV_ENTRY_COMMITTED,
+ NotificationService::AllSources());
+ registrar_.Add(this, NotificationType::TAB_CONTENTS_DESTROYED,
+ NotificationService::AllSources());
+}
+
+void AppModalDialog::ShowModalDialog() {
+ // If the WebContents that created this dialog navigated away before this
+ // dialog became visible, simply show the next dialog if any.
+ if (!web_contents_) {
+ AppModalDialogQueue::ShowNextDialog();
+ delete this;
+ return;
+ }
+
+ web_contents_->Activate();
+ CreateAndShowDialog();
+}
+
+void AppModalDialog::OnCancel() {
+ // We need to do this before WM_DESTROY (WindowClosing()) as any parent frame
+ // will receive it's activation messages before this dialog receives
+ // WM_DESTROY. The parent frame would then try to activate any modal dialogs
+ // that were still open in the ModalDialogQueue, which would send activation
+ // back to this one. The framework should be improved to handle this, so this
+ // is a temporary workaround.
+ AppModalDialogQueue::ShowNextDialog();
+
+ if (web_contents_) {
+ web_contents_->OnJavaScriptMessageBoxClosed(reply_msg_, false,
+ std::wstring());
+ }
+}
+
+void AppModalDialog::OnAccept(const std::wstring& prompt_text,
+ bool suppress_js_messages) {
+ AppModalDialogQueue::ShowNextDialog();
+
+ if (web_contents_) {
+ web_contents_->OnJavaScriptMessageBoxClosed(reply_msg_, true,
+ prompt_text);
+
+ if (suppress_js_messages)
+ web_contents()->set_suppress_javascript_messages(true);
+ }
+}
« no previous file with comments | « chrome/browser/app_modal_dialog.h ('k') | chrome/browser/app_modal_dialog_queue.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698