Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs (Patch set 1 Reverts revert of http://codereview.chromium.org/7670041/)

This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=96935
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.


Review URL: http://codereview.chromium.org/7957002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102071 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
This commit is contained in:
flackr@chromium.org 2011-09-21 02:03:38 +00:00
Родитель ef323b60a0
Коммит a16a345354
1 изменённых файлов: 0 добавлений и 16 удалений

Просмотреть файл

@ -34,11 +34,6 @@
# Disable touch support by default.
'touchui%': 0,
# Disable webui dialog replacements for native dialogs by default.
# TODO(flackr): Change this to a runtime flag triggered by
# --pure-views so that these dialogs can be easily tested.
'webui_dialogs%': 0,
# Whether the compositor is enabled on views.
'views_compositor%': 0,
@ -49,7 +44,6 @@
'chromeos%': '<(chromeos)',
'use_only_pure_views%': '<(use_only_pure_views)',
'touchui%': '<(touchui)',
'webui_dialogs%': '<(webui_dialogs)',
'views_compositor%': '<(views_compositor)',
'use_aura%': '<(use_aura)',
@ -80,11 +74,6 @@
'use_only_pure_views%': 0,
}],
# Use WebUI dialogs in TouchUI and PureView builds.
['touchui==1 or use_only_pure_views==1 or use_aura==1', {
'webui_dialogs%': 1,
}],
# Use the views compositor when using the Aura window manager.
['use_aura==1', {
'views_compositor%': 1,
@ -95,7 +84,6 @@
# Copy conditionally-set variables out one scope.
'chromeos%': '<(chromeos)',
'touchui%': '<(touchui)',
'webui_dialogs%': '<(webui_dialogs)',
'host_arch%': '<(host_arch)',
'toolkit_views%': '<(toolkit_views)',
'use_only_pure_views%': '<(use_only_pure_views)',
@ -322,7 +310,6 @@
'enable_flapper_hacks%': '<(enable_flapper_hacks)',
'chromeos%': '<(chromeos)',
'touchui%': '<(touchui)',
'webui_dialogs%': '<(webui_dialogs)',
'file_manager_extension%': '<(file_manager_extension)',
'webui_task_manager%': '<(webui_task_manager)',
'inside_chromium_build%': '<(inside_chromium_build)',
@ -754,9 +741,6 @@
['touchui==1', {
'grit_defines': ['-D', 'touchui'],
}],
['webui_dialogs==1', {
'grit_defines': ['-D', 'webui_dialogs'],
}],
['file_manager_extension==1', {
'grit_defines': ['-D', 'file_manager_extension'],
}],