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

Unified Diff: chrome/browser/ui/views/native_web_contents_modal_dialog_manager_views.h

Issue 2087643003: Move web modal dialog manager files into the constrained_window component. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Revert changes to gyp files, IWYU for #ifdef (OS_MACOSX). Created 4 years, 4 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/ui/views/native_web_contents_modal_dialog_manager_views.h
diff --git a/chrome/browser/ui/views/native_web_contents_modal_dialog_manager_views.h b/chrome/browser/ui/views/native_web_contents_modal_dialog_manager_views.h
deleted file mode 100644
index 21b2b9b3bf31f661d43accca3ac819642146bcde..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/native_web_contents_modal_dialog_manager_views.h
+++ /dev/null
@@ -1,76 +0,0 @@
-// Copyright 2016 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.
-
-#ifndef CHROME_BROWSER_UI_VIEWS_NATIVE_WEB_CONTENTS_MODAL_DIALOG_MANAGER_VIEWS_H_
-#define CHROME_BROWSER_UI_VIEWS_NATIVE_WEB_CONTENTS_MODAL_DIALOG_MANAGER_VIEWS_H_
-
-#include <set>
-
-#include "base/macros.h"
-#include "components/web_modal/modal_dialog_host.h"
-#include "components/web_modal/single_web_contents_dialog_manager.h"
-#include "ui/gfx/native_widget_types.h"
-#include "ui/views/widget/widget_observer.h"
-
-namespace views {
-class Widget;
-}
-
-// Class for parenting a tab-modal views dialog off of a views browser.
-class NativeWebContentsModalDialogManagerViews
- : public web_modal::SingleWebContentsDialogManager,
- public web_modal::ModalDialogHostObserver,
- public views::WidgetObserver {
- public:
- NativeWebContentsModalDialogManagerViews(
- gfx::NativeWindow dialog,
- web_modal::SingleWebContentsDialogManagerDelegate* native_delegate);
-
- ~NativeWebContentsModalDialogManagerViews() override;
-
- // Sets up this object to manage the |dialog_|. Registers for closing events
- // in order to notify the delegate.
- void ManageDialog();
-
- // web_modal::SingleWebContentsDialogManager:
- void Show() override;
- void Hide() override;
- void Close() override;
- void Focus() override;
- void Pulse() override;
-
- // web_modal::ModalDialogHostObserver:
- void OnPositionRequiresUpdate() override;
- void OnHostDestroying() override;
-
- // views::WidgetObserver:
-
- // NOTE(wittman): OnWidgetClosing is overriden to ensure that, when the widget
- // is explicitly closed, the destruction occurs within the same call
- // stack. This avoids event races that lead to non-deterministic destruction
- // ordering in e.g. the print preview dialog. OnWidgetDestroying is overridden
- // because OnWidgetClosing is *only* invoked on explicit close, not when the
- // widget is implicitly destroyed due to its parent being closed. This
- // situation occurs with app windows. WidgetClosing removes the observer, so
- // only one of these two functions is ever invoked for a given widget.
- void OnWidgetClosing(views::Widget* widget) override;
- void OnWidgetDestroying(views::Widget* widget) override;
- void HostChanged(web_modal::WebContentsModalDialogHost* new_host) override;
- gfx::NativeWindow dialog() override;
-
- private:
- static views::Widget* GetWidget(gfx::NativeWindow dialog);
- void WidgetClosing(views::Widget* widget);
-
- web_modal::SingleWebContentsDialogManagerDelegate* native_delegate_;
- gfx::NativeWindow dialog_;
- web_modal::WebContentsModalDialogHost* host_;
- bool host_destroying_;
- std::set<views::Widget*> observed_widgets_;
- std::set<views::Widget*> shown_widgets_;
-
- DISALLOW_COPY_AND_ASSIGN(NativeWebContentsModalDialogManagerViews);
-};
-
-#endif // CHROME_BROWSER_UI_VIEWS_NATIVE_WEB_CONTENTS_MODAL_DIALOG_MANAGER_VIEWS_H_

Powered by Google App Engine
This is Rietveld 408576698