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

Unified Diff: chrome/browser/ui/views/app_list/app_list_dialog_contents_view.cc

Issue 405873003: Changed AppListDialogContentsView to extend DialogDelegate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 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/app_list/app_list_dialog_contents_view.cc
diff --git a/chrome/browser/ui/views/app_list/app_list_dialog_contents_view.cc b/chrome/browser/ui/views/app_list/app_list_dialog_contents_view.cc
index 3378bdfe1487541b298158e0bf1c2d7010b3f277..3b1f36fed3c3d1f6045dde8da699e181b9e35e51 100644
--- a/chrome/browser/ui/views/app_list/app_list_dialog_contents_view.cc
+++ b/chrome/browser/ui/views/app_list/app_list_dialog_contents_view.cc
@@ -154,6 +154,10 @@ ui::ModalType AppListDialogContentsView::GetModalType() const {
return ui::MODAL_TYPE_WINDOW;
}
+bool AppListDialogContentsView::IsDialogBox() {
+ return true;
tapted 2014/07/21 00:34:07 It looks like you'll also pick up "WS_EX_DLGMODALF
sashab 2014/07/21 03:36:20 Not at all. Enabling it doesn't seem to do anythin
+}
+
ui::AXRole AppListDialogContentsView::GetAccessibleWindowRole() const {
return ui::AX_ROLE_DIALOG;
}

Powered by Google App Engine
This is Rietveld 408576698