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

Unified Diff: chrome/chrome_exe.gypi

Issue 107663008: Chrome browser process DLL blacklist. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Greg's feedback. Created 7 years 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_exe.gypi
diff --git a/chrome/chrome_exe.gypi b/chrome/chrome_exe.gypi
index abd63a3c7d6663d050ccd4e083e4ad596112c722..c8f9ceff6911c124e5675586d3dd8b0b305b7b3a 100644
--- a/chrome/chrome_exe.gypi
+++ b/chrome/chrome_exe.gypi
@@ -465,6 +465,8 @@
}],
['OS=="win"', {
'dependencies': [
+ # Note that chrome_elf must be listed first. Do not reorder it.
+ '../chrome_elf/chrome_elf.gyp:chrome_elf',
Sigurður Ásgeirsson 2013/12/17 14:31:01 The DLL dependency list and order here could do wi
robertshield 2013/12/17 14:34:10 Agreed! Tests for this and the other are in a foll
'chrome_dll',
'chrome_nacl_win64',
'chrome_process_finder',
@@ -474,7 +476,6 @@
'../base/base.gyp:base',
'../breakpad/breakpad.gyp:breakpad_handler',
'../breakpad/breakpad.gyp:breakpad_sender',
- '../chrome_elf/chrome_elf.gyp:chrome_elf',
'../components/components.gyp:breakpad_component',
'../components/components.gyp:policy',
'../sandbox/sandbox.gyp:sandbox',

Powered by Google App Engine
This is Rietveld 408576698