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

Unified Diff: chrome/test/BUILD.gn

Issue 2556883002: Remove enable_web_speech flag. (Closed)
Patch Set: Merge Created 4 years 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 | « build/config/features.gni ('k') | content/browser/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index 95347be761a95d3cf46d1cba8d869df0eca9bbf8..33f46f716e6b191eac449dd4cd22c32a990d88ba 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -1287,7 +1287,7 @@ test("browser_tests") {
]
enable_multidex = true
- } else {
+ } else { # Not Android.
sources += [
# The list of sources which is only used by chrome browser tests on
# desktop platforms.
@@ -1696,6 +1696,11 @@ test("browser_tests") {
"../browser/sessions/session_restore_browsertest_chromeos.cc",
"../browser/sessions/tab_restore_browsertest.cc",
"../browser/site_details_browsertest.cc",
+
+ # If this list is used on Android in the future, these browser/speech/*
+ # files will probably not be applicable.
+ "../browser/speech/extension_api/tts_extension_apitest.cc",
+ "../browser/speech/speech_recognition_browsertest.cc",
"../browser/spellchecker/spellcheck_service_browsertest.cc",
"../browser/ssl/captive_portal_blocking_page_browsertest.cc",
"../browser/ssl/cert_verifier_browser_test.cc",
@@ -2372,12 +2377,6 @@ test("browser_tests") {
[ "../browser/ui/views/ime/input_ime_apitest_nonchromeos.cc" ]
}
}
- if (enable_web_speech) {
- sources += [
- "../browser/speech/extension_api/tts_extension_apitest.cc",
- "../browser/speech/speech_recognition_browsertest.cc",
- ]
- }
if (safe_browsing_mode == 1) {
sources += [
"../browser/safe_browsing/permission_reporter_browsertest.cc",
@@ -3608,6 +3607,8 @@ test("unit_tests") {
# GCMDriverAndroid is not yet implemented.
"../browser/services/gcm/gcm_profile_service_unittest.cc",
"../browser/sessions/persistent_tab_restore_service_unittest.cc",
+ "../browser/speech/extension_api/extension_manifests_tts_unittest.cc",
+ "../browser/speech/tts_controller_unittest.cc",
"../browser/sync/sessions/sessions_sync_manager_unittest.cc",
"../browser/sync/sync_ui_util_unittest.cc", # Sync setup uses native ui.
"../browser/translate/translate_manager_render_view_host_unittest.cc",
@@ -4198,12 +4199,6 @@ test("unit_tests") {
"../browser/printing/cloud_print/privet_url_fetcher_unittest.cc",
]
}
- if (enable_web_speech) {
- sources += [
- "../browser/speech/extension_api/extension_manifests_tts_unittest.cc",
- "../browser/speech/tts_controller_unittest.cc",
- ]
- }
if (safe_browsing_mode > 0) {
sources += [
« no previous file with comments | « build/config/features.gni ('k') | content/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698