Add a WinHeap dump provider to the memory profiler.

base/trace_event/trace_event.gypi

 # Copyright 2015 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.
 {
   'variables': {
     'trace_event_sources' : [
       'trace_event/memory_allocator_attributes_type_info.cc',
       'trace_event/memory_allocator_attributes_type_info.h',
       'trace_event/memory_allocator_dump.cc',
       'trace_event/memory_allocator_dump.h',
@@ skipping 32 matching lines @@
       'trace_event/trace_event_win.cc',
       'trace_event/trace_event_win.h',
     ],
     'conditions': [
       ['OS == "linux" or OS == "android"', {
           'trace_event_sources': [
             'trace_event/malloc_dump_provider.cc',
             '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 suffix.
Unforuntately, as you might have found out, you cannot benefit of the auto
exclusion rule (which excludes *win.cc on Non-win platforms) if you are in a
gypi file.
...at which point i'd just make call just winheap_dump_provider.

[Sébastien Marchand, 2015/04/23 10:52:02]

It looks like I can just use the _win suffix and get rid of the condition
(https://code.google.com/p/gyp/wiki/GypUserDocumentation#Add_a_platform-specif...)

+            'trace_event/winheap_dump_provider.h',
+          ],
+      }],
     ],
     'trace_event_test_sources' : [
       'trace_event/memory_allocator_attributes_type_info_unittest.cc',
       'trace_event/memory_allocator_dump_unittest.cc',
       'trace_event/memory_dump_manager_unittest.cc',
       'trace_event/process_memory_maps_dump_provider_unittest.cc',
       'trace_event/process_memory_totals_dump_provider_unittest.cc',
       'trace_event/trace_event_argument_unittest.cc',
       'trace_event/trace_event_memory_unittest.cc',
       'trace_event/trace_event_synthetic_delay_unittest.cc',
       '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? (or just move the conditions
block above in lines 46-58 to the end, so you can use the same OS==win condition
to augment both trace_event_sources and trace_event_Test_sources

[Sébastien Marchand, 2015/04/23 10:52:02]

Ditto.

     ],
   },
 }