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

Unified Diff: build/common.gypi

Issue 461633002: Refactor language detection logic to allow non-static CLD data sources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix linux/mac/windows compile issues Created 6 years, 2 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
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 6301aebbfe145bd3c163fdc6a40f17b619faaf17..06075e66ba3a57d56e521a4b5e706881638cae76 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -505,21 +505,6 @@
# 2: Large tables, high accuracy
'cld2_table_size%': 2,
- # The data acquisition mode for CLD2. Possible values are:
- # static: CLD2 data is statically linked to the executable.
- # standalone: CLD2 data is provided in a standalone file that is
- # bundled with the executable.
- # component: CLD2 data is provided as a Chrome "component" and is
- # downloaded via the component updater.
- #
- # For more information on switching the CLD2 data source, see:
- # https://sites.google.com/a/chromium.org/dev/developers/how-tos/compact-language-detector-cld-data-source-configuration
- #
- # This string will be exposed in chrome://translate-internals under the
- # heading "CLD Data Source". This allows easy determination of which
- # data source the browser was built with.
- 'cld2_data_source%': 'static',
-
# Enable spell checker.
'enable_spellcheck%': 1,
@@ -758,7 +743,7 @@
['OS=="android"', {
'enable_extensions%': 0,
'enable_google_now%': 0,
- 'cld_version%': 1,
+ 'cld_version%': 2,
'enable_spellcheck%': 0,
'enable_themes%': 0,
'remoting%': 0,
@@ -1178,7 +1163,6 @@
'enable_google_now%': '<(enable_google_now)',
'cld_version%': '<(cld_version)',
'cld2_table_size%': '<(cld2_table_size)',
- 'cld2_data_source%': '<(cld2_data_source)',
'enable_captive_portal_detection%': '<(enable_captive_portal_detection)',
'disable_file_support%': '<(disable_file_support)',
'disable_ftp_support%': '<(disable_ftp_support)',
@@ -2908,11 +2892,6 @@
['cld_version!=0', {
'defines': ['CLD_VERSION=<(cld_version)'],
}],
- ['cld_version==2', {
- # This is used to populate the "CLD Data Source" field in:
- # chrome://translate-internals
- 'defines': ['CLD2_DATA_SOURCE=<(cld2_data_source)'],
- }],
['enable_printing==1', {
'defines': ['ENABLE_FULL_PRINTING=1', 'ENABLE_PRINTING=1'],
}],
« no previous file with comments | « no previous file | build/config/features.gni » ('j') | chrome/browser/component_updater/cld_component_installer.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698