[tracing] Fix PartitionAlloc dumper reporting

Source/wtf/Partitions.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 100 matching lines @@
         observedMaxSizeInMB = sizeInMB;
     }
 }
 
 void Partitions::dumpMemoryStats(bool isLightDump, PartitionStatsDumper* partitionStatsDumper)
 {
     // Object model and rendering partitions are not thread safe and can be
     // accessed only on the main thread.
     ASSERT(isMainThread());
 
-    partitionDumpStatsGeneric(fastMallocPartition(), "fast_malloc_partition", isLightDump, partitionStatsDumper);
-    partitionDumpStatsGeneric(bufferPartition(), "buffer_partition", isLightDump, partitionStatsDumper);
-    partitionDumpStats(nodePartition(), "node_partition", isLightDump, partitionStatsDumper);
-    partitionDumpStats(layoutPartition(), "layout_partition", isLightDump, partitionStatsDumper);
+    partitionDumpStatsGeneric(fastMallocPartition(), "fast_malloc", isLightDump, partitionStatsDumper);

[haraken, 2015/09/07 23:24:26]

I'd keep "_partition" for clarity.

[Primiano Tucci (use gerrit), 2015/09/08 07:56:56]

I did remove it because the dump path is now:

partition_alloc/partitions/fast_malloc_partition etc I felt it was a bit
redundant having "partition" twice in the name.
WDYT? Restored in the meanwhile

[haraken, 2015/09/08 08:07:33]

ah, makes sense. Let's drop "_partition".

+    partitionDumpStatsGeneric(bufferPartition(), "buffer", isLightDump, partitionStatsDumper);
+    partitionDumpStats(nodePartition(), "node", isLightDump, partitionStatsDumper);
+    partitionDumpStats(layoutPartition(), "layout", isLightDump, partitionStatsDumper);
 }
 
 static NEVER_INLINE void partitionsOutOfMemoryUsing2G()
 {
     size_t signature = 2UL * 1024 * 1024 * 1024;
     alias(&signature);
     IMMEDIATE_CRASH();
 }
 
 static NEVER_INLINE void partitionsOutOfMemoryUsing1G()
@@ skipping 69 matching lines @@
     if (totalUsage >= 64 * 1024 * 1024)
         partitionsOutOfMemoryUsing64M();
     if (totalUsage >= 32 * 1024 * 1024)
         partitionsOutOfMemoryUsing32M();
     if (totalUsage >= 16 * 1024 * 1024)
         partitionsOutOfMemoryUsing16M();
     partitionsOutOfMemoryUsingLessThan16M();
 }
 
 } // namespace WTF