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

Side by Side Diff: base/files/protect_file_posix.gypi

Issue 676873004: Intercept file Open/Close (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: disable linker magic for component build Created 6 years, 1 month 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
(Empty)
1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 # Provides sanity-checks and early crashes on some improper use of posix file
6 # descriptors. See protect_file_posix.cc for details.
7 #
8 # Usage:
9 # {
10 # 'target_name': 'libsomething',
11 # 'type': 'shared_library', // Do *not* use it for static libraries.
12 # 'includes': [
13 # 'base/files/protect_file_posix.gypi'
14 # ],
15 # ...
16 # }
17 #
18 # TODO(pasko): GN.
19 {
20 'conditions': [
21 # In the component build the interceptors have to be declared with
22 # non-hidden visibility, which is not desirable for the release build.
23 # Disable the extra checks for the component build for simplicity.
24 ['component != "shared_library"', {
Fabrice (no longer in Chrome) 2014/11/04 15:37:21 Maybe you need to add a test for Windows here if y
pasko 2014/11/04 17:48:06 I am hoping that my scary comments would prevent p
25 'ldflags': [
26 '-Wl,--wrap=close',
27 ],
28 'dependencies': [
29 '<(DEPTH)/base/base.gyp:protect_file_posix',
30 ],
31 }],
32 ],
33 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698