diff options
| author | scottmg@chromium.org <scottmg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-16 07:45:36 +0000 |
|---|---|---|
| committer | scottmg@chromium.org <scottmg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-16 07:45:36 +0000 |
| commit | d7a2d89aee556017139f6bea76e328423e60277d (patch) | |
| tree | 018c1358276c3c551e888568f1a0e19ab840da25 /build/common.gypi | |
| parent | ed522c341e491b8aad34f3ac2d9c06669971627e (diff) | |
| download | chromium_src-d7a2d89aee556017139f6bea76e328423e60277d.zip chromium_src-d7a2d89aee556017139f6bea76e328423e60277d.tar.gz chromium_src-d7a2d89aee556017139f6bea76e328423e60277d.tar.bz2 | |
Set up content in-process main threads via factory
This code was previously #ifdef'd out based on CHROME_MULTIPLE_DLL. This works
for chrome, but not for test targets which link content_browser. content_browser
needs to not link against child-only targets (as they'll cause linking blink
into the browser dll).
Instead of having utility_process_host_impl, et al. own the in-process
implementation, use a factory to create them that's installed in test code, and
in chrome for supporting --single-process.
At the same time, remove the global CHROME_MULTIPLE_DLL define and localize it
to chrome_exe.gypi because it's too easy to use incorrectly.
TBR=darin
R=piman@chromium.org,jam@chromium.org
BUG=237249
Review URL: https://chromiumcodereview.appspot.com/23235002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217968 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'build/common.gypi')
| -rw-r--r-- | build/common.gypi | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/build/common.gypi b/build/common.gypi index e95efab..df25eb4 100644 --- a/build/common.gypi +++ b/build/common.gypi @@ -1899,9 +1899,6 @@ '<(DEPTH)/base/allocator/allocator.gyp:type_profiler', ], }], - ['chrome_multiple_dll', { - 'defines': ['CHROME_MULTIPLE_DLL'], - }], ['OS=="linux" and clang==1 and host_arch=="ia32"', { # TODO(dmikurube): Remove -Wno-sentinel when Clang/LLVM is fixed. # See http://crbug.com/162818. |