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

Unified Diff: chrome/chrome_resources.gyp

Issue 10825234: Revert 150411 - Repack separate 100 / 200 / touch_100 / touch_200 files into single pak files. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 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 | « chrome/chrome_repack_chrome_touch_200_percent.gypi ('k') | chrome/installer/mini_installer/chrome.release » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_resources.gyp
===================================================================
--- chrome/chrome_resources.gyp (revision 150448)
+++ chrome/chrome_resources.gyp (working copy)
@@ -221,6 +221,53 @@
'theme_resources_gen',
'<(DEPTH)/ui/ui.gyp:ui_resources',
],
+ 'conditions': [
+ ['OS != "mac"', {
+ # Copy pak files to the product directory. These files will be picked
+ # up by the following installer scripts:
+ # - Windows: chrome/installer/mini_installer/chrome.release
+ # - Linux: chrome/installer/linux/internal/common/installer.include
+ # Ensure that the above scripts are updated when adding or removing
+ # pak files.
+ # Copying files to the product directory is not needed on the Mac
+ # since the framework build phase will copy them into the framework
+ # bundle directly.
+ 'copies': [
+ {
+ 'destination': '<(PRODUCT_DIR)',
+ 'files': [
+ '<(grit_out_dir)/theme_resources_100_percent.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_resources_100_percent.pak',
+ ],
+ },
+ ],
+ }],
+ ['(OS != "mac" and enable_hidpi == 1) or chromeos == 1', {
+ 'copies': [
+ {
+ 'destination': '<(PRODUCT_DIR)',
+ 'files': [
+ '<(grit_out_dir)/theme_resources_200_percent.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_resources_200_percent.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_resources_200_percent.pak',
+ ],
+ },
+ ],
+ }],
+ ['enable_touch_ui==1', {
+ 'copies': [
+ {
+ 'destination': '<(PRODUCT_DIR)',
+ 'files': [
+ '<(grit_out_dir)/theme_resources_touch_100_percent.pak',
+ '<(grit_out_dir)/theme_resources_touch_200_percent.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_resources_touch_100_percent.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_resources_touch_200_percent.pak',
+ ],
+ },
+ ],
+ }],
+ ],
},
{
'target_name': 'packed_extra_resources',
@@ -283,30 +330,11 @@
{
'includes': ['chrome_repack_pseudo_locales.gypi']
},
- {
- 'includes': ['chrome_repack_chrome_100_percent.gypi']
- },
- {
- 'includes': ['chrome_repack_chrome_200_percent.gypi']
- },
- {
- 'includes': ['chrome_repack_chrome_touch_100_percent.gypi']
- },
- {
- 'includes': ['chrome_repack_chrome_touch_200_percent.gypi']
- },
],
'conditions': [
['OS != "mac"', {
- # Copy pak files to the product directory. These files will be picked
- # up by the following installer scripts:
- # - Windows: chrome/installer/mini_installer/chrome.release
- # - Linux: chrome/installer/linux/internal/common/installer.include
- # Ensure that the above scripts are updated when adding or removing
- # pak files.
- # Copying files to the product directory is not needed on the Mac
- # since the framework build phase will copy them into the framework
- # bundle directly.
+ # We'll install the resource files to the product directory. The Mac
+ # copies the results over as bundle resources in its own special way.
'copies': [
{
'destination': '<(PRODUCT_DIR)',
@@ -315,12 +343,6 @@
],
},
{
- 'destination': '<(PRODUCT_DIR)',
- 'files': [
- '<(SHARED_INTERMEDIATE_DIR)/repack/chrome_100_percent.pak'
- ],
- },
- {
'destination': '<(PRODUCT_DIR)/locales',
'files': [
'<!@pymod_do_main(repack_locales -o -p <(OS) -g <(grit_out_dir) -s <(SHARED_INTERMEDIATE_DIR) -x <(SHARED_INTERMEDIATE_DIR) <(locales))'
@@ -347,36 +369,6 @@
},
],
}],
- ['enable_hidpi == 1 or chromeos == 1', {
- 'copies': [
- {
- 'destination': '<(PRODUCT_DIR)',
- 'files': [
- '<(SHARED_INTERMEDIATE_DIR)/repack/chrome_200_percent.pak',
- ],
- },
- ],
- }],
- ['enable_touch_ui==1', {
- 'copies': [
- {
- 'destination': '<(PRODUCT_DIR)',
- 'files': [
- '<(SHARED_INTERMEDIATE_DIR)/repack/chrome_touch_100_percent.pak',
- ],
- },
- ],
- }],
- ['enable_hidpi == 1 and enable_touch_ui==1', {
- 'copies': [
- {
- 'destination': '<(PRODUCT_DIR)',
- 'files': [
- '<(SHARED_INTERMEDIATE_DIR)/repack/chrome_touch_200_percent.pak',
- ],
- },
- ],
- }],
], # conditions
}], # end OS != "mac"
], # conditions
« no previous file with comments | « chrome/chrome_repack_chrome_touch_200_percent.gypi ('k') | chrome/installer/mini_installer/chrome.release » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698