Skip to content

Commit

Permalink
Add GN interactive UI tests
Browse files Browse the repository at this point in the history 
TBR=dpranke

Review URL: https://codereview.chromium.org/795373005

Cr-Commit-Position: refs/heads/master@{#310040}
  • Loading branch information
brettw authored and Commit bot committed Jan 6, 2015
1 parent c824fbc commit 0287a1a
Show file tree
Hide file tree
Showing 7 changed files with 295 additions and 38 deletions.
1 change: 1 addition & 0 deletions ash/ash_resources.gyp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
{
'targets': [
{
# GN version: //ash/resources
'target_name': 'ash_resources',
'type': 'none',
'variables': {
Expand Down
4 changes: 4 additions & 0 deletions build/config/linux/BUILD.gn
View file
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,10 @@ config("xi") {
libs = [ "Xi" ]
}

config("xtst") {
libs = [ "Xtst" ]
}

config("libresolv") {
libs = [ "resolv" ]
}
Expand Down
68 changes: 36 additions & 32 deletions chrome/chrome_tests.gypi
View file
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,6 @@
'browser/extensions/api/module/module_apitest.cc',
'browser/extensions/api/music_manager_private/music_manager_private_browsertest.cc',
'browser/extensions/api/notification_provider/notification_provider_apitest.cc',
'browser/extensions/api/notifications/notifications_apitest.cc',
'browser/extensions/api/omnibox/omnibox_api_browsertest.cc',
'browser/extensions/api/page_capture/page_capture_apitest.cc',
'browser/extensions/api/permissions/permissions_apitest.cc',
Expand Down Expand Up @@ -480,8 +479,6 @@
'browser/net/proxy_browsertest.cc',
'browser/net/sdch_browsertest.cc',
'browser/net/websocket_browsertest.cc',
'browser/notifications/login_state_notification_blocker_chromeos_browsertest.cc',
'browser/notifications/message_center_notifications_browsertest.cc',
'browser/password_manager/password_manager_browsertest.cc',
'browser/policy/cloud/cloud_policy_browsertest.cc',
'browser/policy/cloud/cloud_policy_manager_browsertest.cc',
Expand Down Expand Up @@ -876,8 +873,6 @@
'browser/ui/omnibox/omnibox_view_browsertest.cc',
'browser/ui/panels/base_panel_browser_test.cc',
'browser/ui/panels/base_panel_browser_test.h',
'browser/ui/panels/detached_panel_browsertest.cc',
'browser/ui/panels/docked_panel_browsertest.cc',
'browser/ui/panels/test_panel_active_state_observer.cc',
'browser/ui/panels/test_panel_active_state_observer.h',
'browser/ui/panels/test_panel_mouse_watcher.cc',
Expand Down Expand Up @@ -977,7 +972,7 @@
'browser/ui/views/toolbar/toolbar_view_interactive_uitest.cc',
],
'chrome_interactive_ui_test_notifications_sources': [
'browser/notifications/login_state_notification_blocker_chromeos_browsertest.cc',
'browser/extensions/api/notifications/notifications_apitest.cc',
'browser/notifications/message_center_notifications_browsertest.cc',
],
'chrome_interactive_ui_test_chromeos_sources': [
Expand Down Expand Up @@ -1455,24 +1450,28 @@
}],
['OS=="linux" and chromeos==0', {
'sources!': [
# TODO(port): Everything here times out. Attempts have been made to
# fix the individual failures, but each time I disable a test from
# these suites, it seems like one or another starts timing out too.
'browser/ui/views/keyboard_access_browsertest.cc', # Views except Mac views.
# TODO(port): This times out. Attempts have been made to fix the
# individual failures, but each time I disable a test from these
# suites, it seems like one or another starts timing out too.

# Note: list duplicated in GN build.
'browser/ui/views/keyboard_access_browsertest.cc',
],
}, { # Everything but desktop Linux.
'sources': [ '<@(chrome_interactive_ui_test_non_desktop_linux_sources)' ],
}],
['use_ash==1', {
'sources': [ '<@(chrome_interactive_ui_test_ash_sources)' ],
}],
['OS != "mac"', {
'sources': [ '<@(chrome_interactive_ui_test_non_mac_sources)' ],
'dependencies': [
'../ash/ash.gyp:ash_test_support',
],
}],
['OS=="mac"', {
'sources!': [
# TODO(tapted): Enable toolkit-views tests on Mac when their
# respective implementations are ported.

# Note: Sources list duplicated in GN build.
'browser/ui/views/keyboard_access_browsertest.cc',
'browser/ui/views/message_center/web_notification_tray_browsertest.cc',
'browser/ui/views/panels/panel_view_browsertest.cc',
Expand All @@ -1483,32 +1482,41 @@
],
# See comment about the same line in chrome/chrome_tests.gypi.
'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-ObjC']},
}], # OS=="mac"
}, { # Non-Mac.
'sources': [ '<@(chrome_interactive_ui_test_non_mac_sources)' ],
}],
['notifications == 1', {
# Common notifications tests.
'sources': [
'browser/extensions/notifications_apitest.cc',
'<@(chrome_interactive_ui_test_notifications_sources)',
],
'conditions': [
['chromeos == 0', {
['chromeos == 1', {
'sources': [
'browser/notifications/login_state_notification_blocker_chromeos_browsertest.cc',
],
}, {
# Non-ChromeOS notifications tests (ChromeOS does not use
# cross-platform panels).
'sources': [
# Note: List duplicated in GN build.
'browser/notifications/notification_browsertest.cc',
'browser/notifications/platform_notification_service_browsertest.cc',
],
}],
],
}],
['OS=="android"', {
'sources!': [
# Android does not use the message center-based Notification system.
'browser/notifications/message_center_notifications_browsertest.cc',
['OS=="android"', {
'sources!': [
# Note: List duplicated in GN build.

# TODO(peter): Enable the Notification browser tests.
'browser/notifications/notification_browsertest.cc',
'browser/notifications/platform_notification_service_browsertest.cc',
]
# Android does not use the message center-based Notification system.
'browser/notifications/message_center_notifications_browsertest.cc',

# TODO(peter): Enable the Notification browser tests.
'browser/notifications/notification_browsertest.cc',
'browser/notifications/platform_notification_service_browsertest.cc',
]
}],
],
}],
['toolkit_views==1', {
'sources': [ '<@(chrome_interactive_ui_test_views_sources)' ],
Expand All @@ -1518,11 +1526,6 @@
'../ui/views/views.gyp:views_test_support',
],
}],
['use_ash==1', {
'dependencies': [
'../ash/ash.gyp:ash_test_support',
],
}],
['use_aura==0 or chromeos==1', {
'sources!': [
'../ui/views/corewm/desktop_capture_controller_unittest.cc',
Expand All @@ -1546,6 +1549,7 @@
'<@(chrome_interactive_ui_test_chromeos_sources)',
],
'sources!': [
# Note: List duplicated in GN build.
'../ui/views/widget/desktop_aura/desktop_window_tree_host_x11_interactive_uitest.cc',
'../ui/views/widget/desktop_aura/x11_topmost_window_finder_interactive_uitest.cc',

Expand All @@ -1566,6 +1570,7 @@
['chromeos==1 and branding=="Chrome"', {
'sources!': [
# These tests are failing on official cros bots. crbug.com/431450.
# Note: list duplicated in GN build.
'browser/ui/views/bookmarks/bookmark_bar_view_test.cc',
],
}],
Expand Down Expand Up @@ -2097,7 +2102,6 @@
'browser/extensions/api/terminal/terminal_private_apitest.cc',
'browser/invalidation/profile_invalidation_provider_factory_browsertest.cc',
'browser/net/nss_context_chromeos_browsertest.cc',
'browser/notifications/login_state_notification_blocker_chromeos_browsertest.cc',
'browser/ui/ash/keyboard_controller_browsertest.cc',
'browser/ui/views/select_file_dialog_extension_browsertest.cc',
'test/data/webui/certificate_viewer_dialog_test.js',
Expand Down
Loading

0 comments on commit 0287a1a

Please sign in to comment.