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

Unified 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: gn 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 side-by-side diff with in-line comments
Download patch
Index: base/files/protect_file_posix.gypi
diff --git a/base/files/protect_file_posix.gypi b/base/files/protect_file_posix.gypi
new file mode 100644
index 0000000000000000000000000000000000000000..849e0a96b4d82387af105474839413a9e9eb578f
--- /dev/null
+++ b/base/files/protect_file_posix.gypi
@@ -0,0 +1,31 @@
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+# Provides sanity-checks and early crashes on some improper use of posix file
+# descriptors. See protect_file_posix.cc for details.
+#
+# Usage:
+# {
+# 'target_name': 'libsomething',
+# 'type': 'shared_library', // Do *not* use it for static libraries.
+# 'includes': [
+# 'base/files/protect_file_posix.gypi'
Fabrice (no longer in Chrome) 2014/11/17 16:10:54 Nit: add a comma at the end of the line
pasko 2014/11/17 16:25:57 Done.
+# ],
+# ...
+# }
+{
+ 'conditions': [
+ # In the component build the interceptors have to be declared with
+ # non-hidden visibility, which is not desirable for the release build.
+ # Disable the extra checks for the component build for simplicity.
+ ['component != "shared_library"', {
+ 'ldflags': [
+ '-Wl,--wrap=close',
+ ],
+ 'dependencies': [
+ '<(DEPTH)/base/base.gyp:protect_file_posix',
+ ],
+ }],
+ ],
+}

Powered by Google App Engine
This is Rietveld 408576698