Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 # Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2011 The Chromium Authors. All rights reserved. |
| 2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
| 3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
| 4 { | 4 { |
| 5 'target_defaults': { | 5 'target_defaults': { |
| 6 'variables': { | 6 'variables': { |
| 7 'chrome_dll_target': 0, | 7 'chrome_dll_target': 0, |
| 8 }, | 8 }, |
| 9 'target_conditions': [ | 9 'target_conditions': [ |
| 10 ['chrome_dll_target==1', { | 10 ['chrome_dll_target==1', { |
| (...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 56 }, | 56 }, |
| 57 }], # OS=="win" | 57 }], # OS=="win" |
| 58 ], # conditions | 58 ], # conditions |
| 59 }], | 59 }], |
| 60 ], | 60 ], |
| 61 }, | 61 }, |
| 62 'conditions': [ | 62 'conditions': [ |
| 63 ['OS=="mac" or OS=="win"', { | 63 ['OS=="mac" or OS=="win"', { |
| 64 'targets': [ | 64 'targets': [ |
| 65 { | 65 { |
| 66 'target_name': 'chrome_dll', | 66 'variables': { |
|
M-A Ruel
2011/08/02 15:25:14
I think it warrants a small comment explaining why
Roger McFarlane (Google)
2011/08/03 16:22:28
Done.
| |
| 67 'variables': { | |
| 68 'optimize_with_syzygy%': 0, | |
| 69 }, | |
| 70 'chrome_dll_target': 1, | |
| 71 'conditions' : [ | |
| 72 ['OS=="win" and optimize_with_syzygy==1', { | |
| 73 'dll_target_name': 'original_chrome_dll', | |
| 74 }, { | |
| 75 'dll_target_name': 'chrome_dll', | |
| 76 }], | |
| 77 ], | |
| 78 }, | |
| 79 'target_name': '<(dll_target_name)', | |
| 67 'type': 'shared_library', | 80 'type': 'shared_library', |
| 68 'variables': { | |
| 69 'chrome_dll_target': 1, | |
| 70 }, | |
| 71 'dependencies': [ | 81 'dependencies': [ |
| 72 '<@(chromium_dependencies)', | 82 '<@(chromium_dependencies)', |
| 73 'app/policy/cloud_policy_codegen.gyp:policy', | 83 'app/policy/cloud_policy_codegen.gyp:policy', |
| 74 ], | 84 ], |
| 75 'conditions': [ | 85 'conditions': [ |
| 76 ['OS=="win"', { | 86 ['OS=="win"', { |
| 77 'product_name': 'chrome', | 87 'product_name': 'chrome', |
| 78 'dependencies': [ | 88 'dependencies': [ |
| 79 # On Windows, link the dependencies (libraries) that make | 89 # On Windows, link the dependencies (libraries) that make |
| 80 # up actual Chromium functionality into this .dll. | 90 # up actual Chromium functionality into this .dll. |
| (...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 126 '<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_resources.rc', | 136 '<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_resources.rc', |
| 127 | 137 |
| 128 # TODO(sgk): left-over from pre-gyp build, figure out | 138 # TODO(sgk): left-over from pre-gyp build, figure out |
| 129 # if we still need them and/or how to update to gyp. | 139 # if we still need them and/or how to update to gyp. |
| 130 #'app/check_dependents.bat', | 140 #'app/check_dependents.bat', |
| 131 #'app/chrome.dll.deps', | 141 #'app/chrome.dll.deps', |
| 132 ], | 142 ], |
| 133 'msvs_settings': { | 143 'msvs_settings': { |
| 134 'VCLinkerTool': { | 144 'VCLinkerTool': { |
| 135 'ImportLibrary': '$(OutDir)\\lib\\chrome_dll.lib', | 145 'ImportLibrary': '$(OutDir)\\lib\\chrome_dll.lib', |
| 136 'ProgramDatabaseFile': '$(OutDir)\\chrome_dll.pdb', | 146 # This corresponds to the /PROFILE flag which ensures |
| 147 # the PDB file contains FIXUP information. | |
| 148 'Profile': 'true', | |
| 149 'conditions' : [ | |
| 150 ['optimize_with_syzygy==1', { | |
| 151 'ProgramDatabaseFile': '$(OutDir)\\original\\chrome_dll.pd b', | |
| 152 'OutputFile': '$(OutDir)\\original\chrome.dll' | |
| 153 }, { | |
| 154 'ProgramDatabaseFile': '$(OutDir)\\chrome_dll.pdb', | |
| 155 }], | |
| 156 ], | |
| 137 }, | 157 }, |
| 138 }, | 158 }, |
| 139 }], # OS=="win" | 159 }], # OS=="win" |
| 140 ['OS=="mac"', { | 160 ['OS=="mac"', { |
| 141 # The main browser executable's name is <(mac_product_name). | 161 # The main browser executable's name is <(mac_product_name). |
| 142 # Certain things will get confused if two modules in the | 162 # Certain things will get confused if two modules in the |
| 143 # executable share the same name, so append " Framework" to the | 163 # executable share the same name, so append " Framework" to the |
| 144 # product_name used for the framework. This will result in | 164 # product_name used for the framework. This will result in |
| 145 # a name like "Chromium Framework.framework". | 165 # a name like "Chromium Framework.framework". |
| 146 'product_name': '<(mac_product_name) Framework', | 166 'product_name': '<(mac_product_name) Framework', |
| (...skipping 565 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 712 'LinkIncremental': '<(msvs_debug_link_nonincremental)', | 732 'LinkIncremental': '<(msvs_debug_link_nonincremental)', |
| 713 }, | 733 }, |
| 714 }, | 734 }, |
| 715 }, | 735 }, |
| 716 }, | 736 }, |
| 717 }, # target chrome_dll | 737 }, # target chrome_dll |
| 718 ], | 738 ], |
| 719 }], | 739 }], |
| 720 ], | 740 ], |
| 721 } | 741 } |
| OLD | NEW |