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

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: Fix merge error in chrome/browser/BUILD.gn 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: components/translate.gypi
diff --git a/components/translate.gypi b/components/translate.gypi
index 2be7c32b260b5c2572df056059aff93081886d65..fb44d02bc07e51618bcc7e059c5d9cc46233a572 100644
--- a/components/translate.gypi
+++ b/components/translate.gypi
@@ -138,23 +138,20 @@
],
'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',
+ 'translate/content/browser/static_browser_cld_data_provider_factory.cc',
+ 'translate/content/browser/static_browser_cld_data_provider_factory.h',
+ 'translate/content/browser/static_browser_cld_data_provider.cc',
+ 'translate/content/browser/static_browser_cld_data_provider.h',
],
},
{
@@ -175,30 +172,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 +199,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/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',
+ 'translate/content/renderer/static_renderer_cld_data_provider.cc',
+ 'translate/content/renderer/static_renderer_cld_data_provider.h',
],
'conditions': [
['cld_version==0 or cld_version==2', {
@@ -232,18 +214,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',
- ]},
- ],
],
},
],

Powered by Google App Engine
This is Rietveld 408576698