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

Unified Diff: components/translate.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: Make some of the harness factory methods private Created 6 years, 1 month 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 | « components/components_tests.gyp ('k') | components/translate/content/browser/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/translate.gypi
diff --git a/components/translate.gypi b/components/translate.gypi
index 2be7c32b260b5c2572df056059aff93081886d65..402e878726a3e5b3958e855dd993165abd0d73e4 100644
--- a/components/translate.gypi
+++ b/components/translate.gypi
@@ -108,7 +108,7 @@
'translate/core/language_detection/language_detection_util.h',
],
'conditions': [
- ['cld_version==0 or cld_version==1', {
+ ['cld_version==1', {
'dependencies': [
'<(DEPTH)/third_party/cld/cld.gyp:cld',
],
@@ -138,23 +138,16 @@
],
'sources': [
# Note: sources list duplicated in GN build.
+ 'translate/content/browser/browser_cld_data_provider_factory.cc',
+ 'translate/content/browser/browser_cld_data_provider_factory.h',
+ 'translate/content/browser/browser_cld_data_provider.cc',
'translate/content/browser/browser_cld_data_provider.h',
+ 'translate/content/browser/browser_cld_utils.cc',
+ 'translate/content/browser/browser_cld_utils.h',
'translate/content/browser/content_translate_driver.cc',
'translate/content/browser/content_translate_driver.h',
- ],
- 'conditions': [
- ['cld2_data_source=="standalone" or cld2_data_source=="component"', {
- 'sources': [
- 'translate/content/browser/data_file_browser_cld_data_provider.cc',
- 'translate/content/browser/data_file_browser_cld_data_provider.h',
- ]},
- ],
- ['cld2_data_source=="static"', {
- 'sources': [
- 'translate/content/browser/static_browser_cld_data_provider.cc',
- 'translate/content/browser/static_browser_cld_data_provider.h',
- ]},
- ],
+ 'translate/content/browser/data_file_browser_cld_data_provider.cc',
+ 'translate/content/browser/data_file_browser_cld_data_provider.h',
],
},
{
@@ -175,30 +168,10 @@
# Note: sources list duplicated in GN build.
'translate/content/common/translate_messages.cc',
'translate/content/common/translate_messages.h',
+ 'translate/content/common/cld_data_source.cc',
'translate/content/common/cld_data_source.h',
- ],
- 'conditions': [
- ['cld2_data_source=="standalone" or cld2_data_source=="component"', {
- 'sources': [
- 'translate/content/common/data_file_cld_data_provider_messages.cc',
- 'translate/content/common/data_file_cld_data_provider_messages.h',
- ]},
- ],
- ['cld2_data_source=="standalone"', {
- 'sources': [
- 'translate/content/common/standalone_cld_data_source.cc',
- ]},
- ],
- ['cld2_data_source=="component"', {
- 'sources': [
- 'translate/content/common/component_cld_data_source.cc',
- ]},
- ],
- ['cld2_data_source=="static"', {
- 'sources': [
- 'translate/content/common/static_cld_data_source.cc',
- ]},
- ],
+ 'translate/content/common/data_file_cld_data_provider_messages.cc',
+ 'translate/content/common/data_file_cld_data_provider_messages.h',
],
},
{
@@ -222,9 +195,14 @@
],
'sources': [
# Note: sources list duplicated in GN build.
+ 'translate/content/renderer/renderer_cld_data_provider.cc',
'translate/content/renderer/renderer_cld_data_provider.h',
+ 'translate/content/renderer/renderer_cld_utils.cc',
+ 'translate/content/renderer/renderer_cld_utils.h',
'translate/content/renderer/translate_helper.cc',
'translate/content/renderer/translate_helper.h',
+ 'translate/content/renderer/data_file_renderer_cld_data_provider.cc',
+ 'translate/content/renderer/data_file_renderer_cld_data_provider.h',
],
'conditions': [
['cld_version==0 or cld_version==2', {
@@ -232,18 +210,6 @@
'<(DEPTH)/third_party/cld_2/cld_2.gyp:cld_2',
],
}],
- ['cld2_data_source=="standalone" or cld2_data_source=="component"', {
- 'sources': [
- 'translate/content/renderer/data_file_renderer_cld_data_provider.cc',
- 'translate/content/renderer/data_file_renderer_cld_data_provider.h',
- ]},
- ],
- ['cld2_data_source=="static"', {
- 'sources': [
- 'translate/content/renderer/static_renderer_cld_data_provider.cc',
- 'translate/content/renderer/static_renderer_cld_data_provider.h',
- ]},
- ],
],
},
],
« no previous file with comments | « components/components_tests.gyp ('k') | components/translate/content/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698