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

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: fix clobber build 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
Index: chrome/chrome_resources.gyp
diff --git a/chrome/chrome_resources.gyp b/chrome/chrome_resources.gyp
index 3195eb8d7cbab6793f4a7e68798765581063bcd5..ed3f047d1892c26d5c2b8b80052932fc2af1603e 100644
--- a/chrome/chrome_resources.gyp
+++ b/chrome/chrome_resources.gyp
@@ -186,7 +186,7 @@
'includes': [ '../build/grit_target.gypi' ],
},
{
- 'target_name': 'theme_resources',
+ 'target_name': 'theme_resources_gen',
'type': 'none',
'actions': [
{
@@ -214,6 +214,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': {
@@ -262,8 +304,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',
],

Powered by Google App Engine
This is Rietveld 408576698