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

Unified Diff: components/components_tests.gyp

Issue 258043003: Eliminate components_unittests' dependence on chrome resources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 | « no previous file | components/test/run_all_unittests.cc » ('j') | components/test/run_all_unittests.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/components_tests.gyp
diff --git a/components/components_tests.gyp b/components/components_tests.gyp
index a1f587a29e07e97b8ea7cdb6e90e607afdc36e41..6b5d16aae0b9d5cac34fd7706b6f58b3bd58b34b 100644
--- a/components/components_tests.gyp
+++ b/components/components_tests.gyp
@@ -166,9 +166,6 @@
'dependencies': [
'../base/base.gyp:base_prefs_test_support',
'../base/base.gyp:test_support_base',
- # TODO(blundell): Eliminate this dependency by having
- # components_unittests have its own pakfile. crbug.com/348563
- '../chrome/chrome_resources.gyp:packed_extra_resources',
# TODO(blundell): Eliminate the need for this dependency in code
# that iOS shares. crbug.com/325243
'../content/content_shell_and_tests.gyp:test_support_content',
@@ -249,6 +246,19 @@
# Dependencies of variations
'components.gyp:variations',
],
+ 'actions': [
+ {
+ 'action_name': 'repack_components_pack',
tfarina 2014/04/29 15:58:25 I like this. s/repack_components_pack/repack_compo
tony 2014/04/29 16:36:53 I don't think it matters much either way. If you'
+ 'variables': {
+ 'pak_inputs': [
+ '<(SHARED_INTERMEDIATE_DIR)/components/component_resources.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/components/strings/component_strings_en-US.pak',
tony 2014/04/29 16:36:53 Do you need to include ui_strings* or ui_resources
+ ],
+ 'pak_output': '<(PRODUCT_DIR)/components_unittests_resources.pak',
+ },
+ 'includes': [ '../build/repack_action.gypi' ],
+ },
+ ],
'conditions': [
['OS != "ios"', {
'sources': [
« no previous file with comments | « no previous file | components/test/run_all_unittests.cc » ('j') | components/test/run_all_unittests.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698