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

Unified Diff: chrome/chrome_tests.gypi

Issue 2142983002: Add desktop engagement metrics. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | chrome/chrome_tests_unit.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_tests.gypi
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index d90622dee8807887f26a30bef00f2a595387c516..9e4c250b3ece5f92b6b1c6eecb0e5c97d218b163 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -234,7 +234,7 @@
'browser/extensions/background_xhr_browsertest.cc',
'browser/extensions/browsertest_util_browsertest.cc',
'browser/extensions/chrome_app_api_browsertest.cc',
- 'browser/extensions/chrome_ui_overrides_browsertest.cc',
+ 'browser/extensions/chrome_ui_overrides_browsertest.cc',
'browser/extensions/content_capabilities_browsertest.cc',
'browser/extensions/content_script_apitest.cc',
'browser/extensions/content_security_policy_apitest.cc',
@@ -414,7 +414,6 @@
'browser/profiles/profile_browsertest.cc',
'browser/profiles/profile_list_desktop_browsertest.cc',
'browser/profiles/profile_manager_browsertest.cc',
- 'browser/profiles/profile_statistics_browsertest.cc',
'browser/profiles/profile_window_browsertest.cc',
'browser/push_messaging/push_messaging_browsertest.cc',
'browser/referrer_policy_browsertest.cc',
@@ -886,6 +885,12 @@
'browser/ui/webui/options/chromeos/guest_mode_options_ui_browsertest.cc',
'browser/ui/webui/options/chromeos/shared_options_browsertest.cc',
],
+ 'chrome_browser_tests_non_mobile_non_cros_sources': [
+ # Tests for non mobile and non CrOS (includes Linux, Win, Mac).
+ 'browser/metrics/desktop_engagement/audible_contents_tracker_browsertest.cc',
+ 'browser/metrics/desktop_engagement/chrome_visibility_observer_browsertest.cc',
+ 'browser/profiles/profile_statistics_browsertest.cc'
+ ],
'chrome_browser_tests_views_non_cros_sources': [
# This should be brought up on OSX Views but not CrOS.
'browser/ui/views/profiles/profile_chooser_view_browsertest.cc',
@@ -2637,10 +2642,8 @@
'browser/extensions/api/networking_private/networking_private_chromeos_apitest.cc',
],
}],
- ['OS=="android" or OS=="ios" or chromeos == 1', {
- 'sources!': [
- 'browser/profiles/profile_statistics_browsertest.cc',
- ],
+ ['OS=="win" or OS=="mac" or (OS=="linux" and chromeos==0)', {
+ 'sources': [ '<@(chrome_browser_tests_non_mobile_non_cros_sources)' ],
}],
['toolkit_views==1', {
'sources': [ '<@(chrome_browser_tests_views_sources)' ],
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | chrome/chrome_tests_unit.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698