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

Unified Diff: components/app_modal/app_modal_dialog.cc

Issue 2901583002: Fold AppModalDialog into its only subclass, JavaScriptAppModalDialog. (Closed)
Patch Set: fix collapse Created 3 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
« no previous file with comments | « components/app_modal/app_modal_dialog.h ('k') | components/app_modal/app_modal_dialog_queue.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/app_modal/app_modal_dialog.cc
diff --git a/components/app_modal/app_modal_dialog.cc b/components/app_modal/app_modal_dialog.cc
deleted file mode 100644
index 8689b7fc03a199bdde4a33b8cb146113ec0c88a1..0000000000000000000000000000000000000000
--- a/components/app_modal/app_modal_dialog.cc
+++ /dev/null
@@ -1,80 +0,0 @@
-// Copyright (c) 2011 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 "components/app_modal/app_modal_dialog.h"
-
-#include "base/logging.h"
-#include "base/run_loop.h"
-#include "components/app_modal/app_modal_dialog_queue.h"
-#include "components/app_modal/native_app_modal_dialog.h"
-
-using content::WebContents;
-
-namespace app_modal {
-namespace {
-
-AppModalDialogObserver* app_modal_dialog_observer = NULL;
-
-} // namespace
-
-AppModalDialogObserver::AppModalDialogObserver() {
- DCHECK(!app_modal_dialog_observer);
- app_modal_dialog_observer = this;
-}
-
-AppModalDialogObserver::~AppModalDialogObserver() {
- DCHECK(app_modal_dialog_observer);
- app_modal_dialog_observer = NULL;
-}
-
-AppModalDialog::AppModalDialog(WebContents* web_contents,
- const base::string16& title)
- : title_(title),
- completed_(false),
- valid_(true),
- native_dialog_(NULL),
- web_contents_(web_contents) {
-}
-
-AppModalDialog::~AppModalDialog() {
- CompleteDialog();
-}
-
-void AppModalDialog::ShowModalDialog() {
- native_dialog_ = CreateNativeDialog();
- native_dialog_->ShowAppModalDialog();
- if (app_modal_dialog_observer)
- app_modal_dialog_observer->Notify(this);
-}
-
-bool AppModalDialog::IsValid() {
- return valid_;
-}
-
-void AppModalDialog::Invalidate() {
- valid_ = false;
-}
-
-bool AppModalDialog::IsJavaScriptModalDialog() {
- return false;
-}
-
-void AppModalDialog::ActivateModalDialog() {
- DCHECK(native_dialog_);
- native_dialog_->ActivateAppModalDialog();
-}
-
-void AppModalDialog::CloseModalDialog() {
- DCHECK(native_dialog_);
- native_dialog_->CloseAppModalDialog();
-}
-
-void AppModalDialog::CompleteDialog() {
- if (!completed_) {
- completed_ = true;
- AppModalDialogQueue::GetInstance()->ShowNextDialog();
- }
-}
-
-} // namespace app_modal
« no previous file with comments | « components/app_modal/app_modal_dialog.h ('k') | components/app_modal/app_modal_dialog_queue.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698