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

Unified Diff: chrome/browser/ui/views/constrained_window_views.cc

Issue 11647017: Rename platform-independent ConstrainedWindow types to WebContentsModalDialog types (part 1) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: stage changes to preserve history Created 8 years 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/constrained_window_views.cc
diff --git a/chrome/browser/ui/views/constrained_window_views.cc b/chrome/browser/ui/views/constrained_window_views.cc
index fa4b931e13bbcf2e724643cfe29ae5dcb54cefc3..6e1908bec760e8def23f677bd2508b07d89bc974 100644
--- a/chrome/browser/ui/views/constrained_window_views.cc
+++ b/chrome/browser/ui/views/constrained_window_views.cc
@@ -383,7 +383,7 @@ void ConstrainedWindowFrameView::OnThemeChanged() {
void ConstrainedWindowFrameView::ButtonPressed(
views::Button* sender, const ui::Event& event) {
if (sender == close_button_)
- container_->CloseConstrainedWindow();
+ container_->CloseWebContentsModalDialog();
}
int ConstrainedWindowFrameView::NonClientBorderThickness() const {
@@ -549,7 +549,7 @@ class ConstrainedWindowFrameViewAsh : public ash::CustomFrameViewAsh {
virtual void ButtonPressed(views::Button* sender,
const ui::Event& event) OVERRIDE {
if (sender == close_button())
- container_->CloseConstrainedWindow();
+ container_->CloseWebContentsModalDialog();
}
private:
@@ -586,7 +586,7 @@ ConstrainedWindowViews::ConstrainedWindowViews(
ConstrainedWindowTabHelper* constrained_window_tab_helper =
ConstrainedWindowTabHelper::FromWebContents(web_contents_);
- constrained_window_tab_helper->AddConstrainedDialog(this);
+ constrained_window_tab_helper->AddDialog(this);
#if defined(USE_ASH)
GetNativeWindow()->SetProperty(ash::kConstrainedWindowKey, true);
views::corewm::SetModalParent(GetNativeWindow(),
@@ -597,12 +597,12 @@ ConstrainedWindowViews::ConstrainedWindowViews(
ConstrainedWindowViews::~ConstrainedWindowViews() {
}
-void ConstrainedWindowViews::ShowConstrainedWindow() {
+void ConstrainedWindowViews::ShowWebContentsModalDialog() {
Show();
- FocusConstrainedWindow();
+ FocusWebContentsModalDialog();
}
-void ConstrainedWindowViews::CloseConstrainedWindow() {
+void ConstrainedWindowViews::CloseWebContentsModalDialog() {
#if defined(USE_ASH)
gfx::NativeView view = web_contents_->GetNativeView();
// Allow the parent to animate again.
@@ -613,7 +613,7 @@ void ConstrainedWindowViews::CloseConstrainedWindow() {
Close();
}
-void ConstrainedWindowViews::FocusConstrainedWindow() {
+void ConstrainedWindowViews::FocusWebContentsModalDialog() {
ConstrainedWindowTabHelper* helper =
ConstrainedWindowTabHelper::FromWebContents(web_contents_);
if ((!helper->delegate() ||
« no previous file with comments | « chrome/browser/ui/views/constrained_window_views.h ('k') | chrome/browser/ui/views/constrained_window_views_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698