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

Unified Diff: chrome/chrome_resources.gyp

Issue 10024050: Metro/HiDPI: Move 1x icons into separate pak file (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 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 | « chrome/chrome_repack_theme_resources_2x.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
diff --git a/chrome/chrome_resources.gyp b/chrome/chrome_resources.gyp
index 98345787efd95c4363690a22b127e428a1f462e3..18f88fb0a4b503b43e20d1f6c1ee9879545f04cc 100644
--- a/chrome/chrome_resources.gyp
+++ b/chrome/chrome_resources.gyp
@@ -193,7 +193,7 @@
'includes': [ '../build/grit_target.gypi' ],
},
{
- 'target_name': 'theme_resources',
+ 'target_name': 'theme_resources_gen',
'type': 'none',
'actions': [
{
@@ -221,6 +221,48 @@
'includes': [ '../build/grit_target.gypi' ],
},
{
+ 'target_name': 'theme_resources',
+ 'type': 'none',
+ 'dependencies': [
+ 'theme_resources_gen',
+ '<(DEPTH)/ui/ui.gyp:ui_resources_2x',
+ '<(DEPTH)/ui/ui.gyp:ui_resources_standard',
+ ],
+ '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.
+ # Coping 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_standard.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources_standard/ui_resources_standard.pak',
+ ],
+ },
+ ],
+ }],
+ ['OS != "mac" and enable_hidpi == 1', {
+ 'copies': [
+ {
+ 'destination': '<(PRODUCT_DIR)',
+ 'files': [
+ '<(grit_out_dir)/theme_resources_2x.pak',
+ '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources_2x/ui_resources_2x.pak',
+ ],
+ },
+ ],
+ }],
+ ],
+ },
+ {
'target_name': 'packed_extra_resources',
'type': 'none',
'variables': {
@@ -269,8 +311,6 @@
'<(DEPTH)/ui/base/strings/ui_strings.gyp:ui_strings',
'<(DEPTH)/ui/ui.gyp:gfx_resources',
'<(DEPTH)/ui/ui.gyp:ui_resources',
- '<(DEPTH)/ui/ui.gyp:ui_resources_2x',
- '<(DEPTH)/ui/ui.gyp:ui_resources_standard',
'<(DEPTH)/webkit/support/webkit_support.gyp:webkit_resources',
'<(DEPTH)/webkit/support/webkit_support.gyp:webkit_strings',
],
« no previous file with comments | « chrome/chrome_repack_theme_resources_2x.gypi ('k') | chrome/installer/mini_installer/chrome.release » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698