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

Unified Diff: tools/android/heap_profiler/heap_profiler.gyp

Issue 323893002: [Android] Introduce libheap_profiler for memory profiling. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: s/hdump/heap_dump/, simplify JSON, refactor heap_dump main Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: tools/android/heap_profiler/heap_profiler.gyp
diff --git a/tools/android/heap_profiler/heap_profiler.gyp b/tools/android/heap_profiler/heap_profiler.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..89f1d786c46a1c925f58793b48776e9bfc2afff3
--- /dev/null
+++ b/tools/android/heap_profiler/heap_profiler.gyp
@@ -0,0 +1,70 @@
+# 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.
+
+{
+ 'targets': [
+ {
+ # libheap_profiler is the library which will be preloaded in the Zygote /
pasko 2014/06/10 16:59:52 nit: you could mention LD_PRELOAD here to be more
Primiano Tucci (use gerrit) 2014/06/19 12:27:17 Done.
+ # target process and contains the black magic to hook malloc/mmap calls.
+ 'target_name': 'heap_profiler',
+ 'type': 'shared_library',
+ 'sources': [
+ 'heap_profiler_hooks_android.c',
+ ],
+ 'include_dirs': ['../../..'],
+ 'dependencies': ['heap_profiler_core'],
+ },
+ {
+ # heap_profiler_core contains only the tracking metadata code without any
+ # hooks. It is required by both the hprof library itself and the unittest.
+ 'target_name': 'heap_profiler_core',
+ 'type': 'static_library',
+ 'sources': [
+ 'heap_profiler.c',
+ 'heap_profiler.h',
+ ],
+ 'include_dirs': ['../../..'],
+ },
+ {
+ 'target_name': 'heap_dump',
pasko 2014/06/10 16:59:52 nit: change mentions of "hdump" in the commit mess
Primiano Tucci (use gerrit) 2014/06/19 12:27:17 Done.
pasko 2014/06/20 22:07:17 really done? I meant the commit message too :)
Primiano Tucci (use gerrit) 2014/06/23 14:00:32 OMG, sorry, I renamed everything but the commit me
+ 'type': 'executable',
+ 'sources': ['heap_dump.c'],
+ 'include_dirs': ['../../..'],
+ },
+ {
+ 'target_name': 'heap_profiler_unittest',
+ 'type': '<(gtest_target_type)',
+ 'sources': ['heap_profiler_unittest.cc'],
+ 'dependencies': [
+ 'heap_profiler_core',
+ '../../../testing/gtest.gyp:gtest',
+ ],
+ 'include_dirs': ['../../..'],
+ 'conditions': [
+ ['OS == "android"',
+ {
+ 'dependencies': [
+ '../../../testing/android/native_test.gyp:native_test_native_code',
+ ],
+ }
+ ],
+ ]
+ },
+ ],
+ 'conditions': [
+ ['OS == "android"', {
+ 'targets': [
+ {
+ 'target_name': 'heap_profiler_unittest_apk',
+ 'type': 'none',
+ 'dependencies': ['heap_profiler_unittest'],
+ 'variables': {
+ 'test_suite_name': 'heap_profiler_unittest',
+ },
+ 'includes': [ '../../../build/apk_test.gypi' ],
+ },
+ ],
+ }],
+ ],
+}

Powered by Google App Engine
This is Rietveld 408576698