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

Side by Side Diff: chrome/chrome_dll.gypi

Issue 1798273002: Remove configuration policy from chrome build files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@policy_chrome_defines
Patch Set: merge Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 'conditions': [ 5 'conditions': [
6 # Dummy target to allow chrome to require chrome_dll to build 6 # Dummy target to allow chrome to require chrome_dll to build
7 # without actually linking to the library 7 # without actually linking to the library
8 ['OS=="mac"', { 8 ['OS=="mac"', {
9 'targets': [ 9 'targets': [
10 { 10 {
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 ], 121 ],
122 'dependencies': [ 122 'dependencies': [
123 '<@(chromium_browser_dependencies)', 123 '<@(chromium_browser_dependencies)',
124 'chrome_features.gyp:chrome_common_features', 124 'chrome_features.gyp:chrome_common_features',
125 '../content/content.gyp:content_app_browser', 125 '../content/content.gyp:content_app_browser',
126 ], 126 ],
127 'conditions': [ 127 'conditions': [
128 ['OS=="win"', { 128 ['OS=="win"', {
129 'dependencies': [ 129 'dependencies': [
130 '<(DEPTH)/chrome_elf/chrome_elf.gyp:chrome_elf', 130 '<(DEPTH)/chrome_elf/chrome_elf.gyp:chrome_elf',
131 ],
132 }],
133 ['OS=="win" and configuration_policy==1', {
134 'dependencies': [
135 '<(DEPTH)/components/components.gyp:policy', 131 '<(DEPTH)/components/components.gyp:policy',
136 ], 132 ],
137 }], 133 }],
138 ['use_aura==1', { 134 ['use_aura==1', {
139 'dependencies': [ 135 'dependencies': [
140 '../ui/compositor/compositor.gyp:compositor', 136 '../ui/compositor/compositor.gyp:compositor',
141 ], 137 ],
142 }], 138 }],
143 ['OS=="win" and target_arch=="ia32"', { 139 ['OS=="win" and target_arch=="ia32"', {
144 # Add a dependency to custom import library for user32 delay 140 # Add a dependency to custom import library for user32 delay
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after
356 'CHROME_MULTIPLE_DLL_CHILD', 352 'CHROME_MULTIPLE_DLL_CHILD',
357 ], 353 ],
358 'sources': [ 354 'sources': [
359 '<(SHARED_INTERMEDIATE_DIR)/chrome_version/chrome_dll_version.rc', 355 '<(SHARED_INTERMEDIATE_DIR)/chrome_version/chrome_dll_version.rc',
360 'app/chrome_main.cc', 356 'app/chrome_main.cc',
361 'app/chrome_main_delegate.cc', 357 'app/chrome_main_delegate.cc',
362 'app/chrome_main_delegate.h', 358 'app/chrome_main_delegate.h',
363 ], 359 ],
364 'conditions': [ 360 'conditions': [
365 ['OS=="win"', { 361 ['OS=="win"', {
362 'dependencies': [
363 '<(DEPTH)/components/components.gyp:policy',
364 ],
366 'conditions': [ 365 'conditions': [
367 ['chrome_pgo_phase!=0', { 366 ['chrome_pgo_phase!=0', {
368 # Disable Warning 4702 ("Unreachable code") for the WPO/PGO 367 # Disable Warning 4702 ("Unreachable code") for the WPO/PGO
369 # builds. Probably anything that this would catch that 368 # builds. Probably anything that this would catch that
370 # wouldn't be caught in a normal build isn't going to 369 # wouldn't be caught in a normal build isn't going to
371 # actually be a bug, so the incremental value of C4702 for 370 # actually be a bug, so the incremental value of C4702 for
372 # PGO builds is likely very small. 371 # PGO builds is likely very small.
373 'msvs_disabled_warnings': [ 372 'msvs_disabled_warnings': [
374 4702 373 4702
375 ], 374 ],
(...skipping 16 matching lines...) Expand all
392 }], 391 }],
393 ['chrome_pgo_phase==2', { 392 ['chrome_pgo_phase==2', {
394 'msvs_settings': { 393 'msvs_settings': {
395 'VCLinkerTool': { 394 'VCLinkerTool': {
396 'LinkTimeCodeGeneration': '3', 395 'LinkTimeCodeGeneration': '3',
397 }, 396 },
398 }, 397 },
399 }], 398 }],
400 ] 399 ]
401 }], 400 }],
402 ['OS=="win" and configuration_policy==1', {
403 'dependencies': [
404 '<(DEPTH)/components/components.gyp:policy',
405 ],
406 }],
407 ['configuration_policy==1', {
408 'dependencies': [
409 'policy_path_parser',
Andrew T Wilson (Slow) 2016/03/22 00:01:27 Did we lose the dependency on policy_path_parser?
brettw 2016/03/22 17:17:44 Oh yeah, I got confused because it's listed in the
410 ],
411 }],
412 ['enable_plugins==1', { 401 ['enable_plugins==1', {
413 'dependencies': [ 402 'dependencies': [
414 '../pdf/pdf.gyp:pdf', 403 '../pdf/pdf.gyp:pdf',
415 ], 404 ],
416 }], 405 }],
417 ], 406 ],
418 }, # target chrome_child_dll 407 }, # target chrome_child_dll
419 ], 408 ],
420 }], 409 }],
421 ], 410 ],
422 } 411 }
OLDNEW
« no previous file with comments | « chrome/chrome_common.gypi ('k') | chrome/chrome_exe.gypi » ('j') | chrome/test/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698