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

Unified Diff: chrome/chrome_browser_ui.gypi

Issue 666533007: Move JavaScriptDialogManager, JavascriptAppModalDialogViews to components/app_modal_dialogs (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 6 years, 1 month 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/chrome_browser_ui.gypi
diff --git a/chrome/chrome_browser_ui.gypi b/chrome/chrome_browser_ui.gypi
index f08a5f721b7532f30e9167312f4b4722762eda6d..bc4e14aa0d114fca960081b1771969a5898ad4e0 100644
--- a/chrome/chrome_browser_ui.gypi
+++ b/chrome/chrome_browser_ui.gypi
@@ -71,8 +71,7 @@
'browser/ui/app_list/app_list_util.cc',
'browser/ui/app_list/app_list_util.h',
# All other browser/ui/app_list files go in chrome_browser_ui_app_list_sources.
- 'browser/ui/app_modal_dialogs/javascript_dialog_manager.cc',
- 'browser/ui/app_modal_dialogs/javascript_dialog_manager.h',
+ 'browser/ui/app_modal_dialogs/chrome_javascript_native_dialog_factory.h',
'browser/ui/autofill/autofill_dialog_controller.cc',
'browser/ui/autofill/autofill_dialog_controller.h',
'browser/ui/autofill/autofill_dialog_models.cc',
@@ -1926,6 +1925,7 @@
'browser/ui/views/certificate_viewer_win.cc',
'browser/ui/views/chrome_constrained_window_views_client.cc',
'browser/ui/views/chrome_constrained_window_views_client.h',
+ 'browser/ui/views/chrome_javascript_native_dialog_factory_views.cc',
'browser/ui/views/chrome_views_delegate_chromeos.cc',
'browser/ui/views/chrome_web_dialog_view.cc',
'browser/ui/views/collected_cookies_views.cc',
@@ -2078,8 +2078,6 @@
'browser/ui/views/infobars/infobar_container_view.h',
'browser/ui/views/infobars/infobar_view.cc',
'browser/ui/views/infobars/infobar_view.h',
- 'browser/ui/views/javascript_app_modal_dialog_views.cc',
- 'browser/ui/views/javascript_app_modal_dialog_views.h',
'browser/ui/views/load_complete_listener.cc',
'browser/ui/views/load_complete_listener.h',
'browser/ui/views/location_bar/bubble_icon_view.cc',
@@ -2504,7 +2502,7 @@
'browser/ui/webui/certificate_viewer_webui.cc',
'browser/ui/webui/certificate_viewer_webui.h',
],
- # Desktop Linux. Assume aura/ash/views.
+ # Desktop Linux. Assume aura/ash/views/x11.
'chrome_browser_ui_desktop_linux_sources': [
'browser/ui/views/app_list/linux/app_list_linux.cc',
'browser/ui/views/app_list/linux/app_list_linux.h',
@@ -2517,6 +2515,8 @@
'browser/ui/views/frame/browser_desktop_window_tree_host_x11.h',
'browser/ui/views/frame/native_browser_frame_factory_auralinux.cc',
'browser/ui/views/frame/native_browser_frame_factory_desktop_ash.cc',
+ 'browser/ui/views/javascript_app_modal_dialog_views_x11.cc',
+ 'browser/ui/views/javascript_app_modal_dialog_views_x11.h',
'browser/ui/views/status_icons/status_icon_linux_wrapper.cc',
'browser/ui/views/status_icons/status_icon_linux_wrapper.h',
'browser/ui/webui/help/version_updater_basic.cc',
@@ -3052,6 +3052,11 @@
'../third_party/hunspell/hunspell.gyp:hunspell',
],
}],
+ ['enable_extensions==1', {
+ 'dependencies': [
+ '<(DEPTH)/extensions/components/extensions_components.gyp:javascript_dialog_extensions_client',
+ ],
+ }],
# Temporary fix to break the browser_ui target into smaller chunks so it
# will link with SyzyASan builds.
['OS=="win" and syzyasan==1', {

Powered by Google App Engine
This is Rietveld 408576698