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

Unified Diff: base/trace_event/trace_event.gypi

Issue 1100173002: Add a WinHeap dump provider to the memory profiler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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: base/trace_event/trace_event.gypi
diff --git a/base/trace_event/trace_event.gypi b/base/trace_event/trace_event.gypi
index 5cd8ca4373f19046f82fd8b09d31eff7687aa5d1..c01268757f40d8b6653924ed6e3611ec0d14fc67 100644
--- a/base/trace_event/trace_event.gypi
+++ b/base/trace_event/trace_event.gypi
@@ -50,6 +50,12 @@
'trace_event/malloc_dump_provider.h',
],
}],
+ ['OS == "win"', {
+ 'trace_event_sources': [
+ 'trace_event/winheap_dump_provider_win.cc',
Primiano Tucci (use gerrit) 2015/04/22 23:48:32 I think that you can just drop the final _win suff
Sébastien Marchand 2015/04/23 10:52:02 It looks like I can just use the _win suffix and g
+ 'trace_event/winheap_dump_provider.h',
+ ],
+ }],
],
'trace_event_test_sources' : [
'trace_event/memory_allocator_attributes_type_info_unittest.cc',
@@ -63,6 +69,7 @@
'trace_event/trace_event_system_stats_monitor_unittest.cc',
'trace_event/trace_event_unittest.cc',
'trace_event/trace_event_win_unittest.cc',
+ 'trace_event/winheap_dump_provider_unittest.cc',
Primiano Tucci (use gerrit) 2015/04/22 23:48:32 You probably want the same conditional logic here
Sébastien Marchand 2015/04/23 10:52:02 Ditto.
],
},
}

Powered by Google App Engine
This is Rietveld 408576698