Commit 88e40ff0 authored by siyua's avatar siyua Committed by Commit Bot

[AF Paradise] Make migration dialog block browser

Previously was blocking web contents.

Bug: 897998
Change-Id: Ic55d445d396c2ae172c8fff8baec9dba84a663be
Reviewed-on: https://chromium-review.googlesource.com/c/1351260Reviewed-by: default avatarEvan Stade <estade@chromium.org>
Commit-Queue: Siyu An <siyua@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611890}
parent ffd5fd25
......@@ -301,7 +301,9 @@ LocalCardMigrationDialogView::~LocalCardMigrationDialogView() {}
void LocalCardMigrationDialogView::ShowDialog() {
Init();
constrained_window::ShowWebModalDialogViews(this, web_contents_);
constrained_window::CreateBrowserModalDialogViews(
this, web_contents_->GetTopLevelNativeWindow())
->Show();
}
void LocalCardMigrationDialogView::CloseDialog() {
......@@ -317,9 +319,9 @@ gfx::Size LocalCardMigrationDialogView::CalculatePreferredSize() const {
}
ui::ModalType LocalCardMigrationDialogView::GetModalType() const {
// This should be a modal dialog since we don't want users to lose progress
// in the migration workflow until they are done.
return ui::MODAL_TYPE_CHILD;
// This should be a modal dialog blocking the browser since we don't want
// users to lose progress in the migration workflow until they are done.
return ui::MODAL_TYPE_WINDOW;
}
bool LocalCardMigrationDialogView::ShouldShowCloseButton() const {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment