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

Unified Diff: chrome/browser/lifetime/browser_close_manager_browsertest.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
Index: chrome/browser/lifetime/browser_close_manager_browsertest.cc
diff --git a/chrome/browser/lifetime/browser_close_manager_browsertest.cc b/chrome/browser/lifetime/browser_close_manager_browsertest.cc
index 67f6bb69ec80c2f74c22e116fcffdef0e37ed0cc..d4e0baaa7a1bb318ad7d72161e65f32cd74a5ba9 100644
--- a/chrome/browser/lifetime/browser_close_manager_browsertest.cc
+++ b/chrome/browser/lifetime/browser_close_manager_browsertest.cc
@@ -64,12 +64,10 @@
namespace {
app_modal::NativeAppModalDialog* GetNextDialog() {
- app_modal::AppModalDialog* dialog = ui_test_utils::WaitForAppModalDialog();
- EXPECT_TRUE(dialog->IsJavaScriptModalDialog());
- app_modal::JavaScriptAppModalDialog* js_dialog =
- static_cast<app_modal::JavaScriptAppModalDialog*>(dialog);
- CHECK(js_dialog->native_dialog());
- return js_dialog->native_dialog();
+ app_modal::JavaScriptAppModalDialog* dialog =
+ ui_test_utils::WaitForAppModalDialog();
+ CHECK(dialog->native_dialog());
+ return dialog->native_dialog();
}
// Note: call |PrepareForDialog| on the relevant WebContents or Browser before

Powered by Google App Engine
This is Rietveld 408576698