Adding thread id to PartitionAlloc memory dumps dumps for single process mode and bug fix.

Source/platform/PartitionAllocMemoryDumpProvider.cpp

Index: Source/platform/PartitionAllocMemoryDumpProvider.cpp
diff --git a/Source/platform/PartitionAllocMemoryDumpProvider.cpp b/Source/platform/PartitionAllocMemoryDumpProvider.cpp
index b85390454c8e74c2ff9f4c5a113ff61b436aa651..d0478aa694a46d8cb95ce9ec72e0a0562ba37cbf 100644
--- a/Source/platform/PartitionAllocMemoryDumpProvider.cpp
+++ b/Source/platform/PartitionAllocMemoryDumpProvider.cpp
@@ -8,6 +8,7 @@
 #include "public/platform/WebMemoryAllocatorDump.h"
 #include "public/platform/WebProcessMemoryDump.h"
 #include "wtf/Partitions.h"
+#include "wtf/Threading.h"
 
 namespace blink {
 
@@ -36,9 +37,9 @@ void PartitionStatsDumperImpl::partitionsDumpBucketStats(const char* partitionNa
     ASSERT(memoryStats->isValid);
     String dumpName;
     if (memoryStats->isDirectMap)
-        dumpName = String::format("partition_alloc/%s/directMap_%zu", partitionName, ++m_uid);
+        dumpName = String::format("partition_alloc/thread_%lu/%s/directMap_%lu", static_cast<unsigned long>(WTF::currentThread()), partitionName, ++m_uid);

[haraken, 2015/06/18 17:55:30]

Instead of calling WTF::currentThread() here, can we pass in the thread id to
partitionsDumpBucketStats (or include the thread id in partitionName)?

We want to keep PartitionAlloc an isolated library as much as possible (i.e.,
reduce the dependency on other classes). We have a plan to port PartitionAlloc
to Chromium.

[ssid, 2015/06/18 20:55:24]

This module depends on Partitions.h, and Partitions.cpp already works with
threading.
I have added the id as uint64_t since some platforms define ThreadIdentifier as
unsigned long int and that is the biggest type.

Passing around a String means PartitionAlloc should depend on WTF::String now
since it has to use String::format. So I just passed the id and converting it to
string at the provider.

     else
-        dumpName = String::format("partition_alloc/%s/bucket_%zu", partitionName, static_cast<size_t>(memoryStats->bucketSlotSize));
+        dumpName = String::format("partition_alloc/thread_%lu/%s/bucket_%u", static_cast<unsigned long>(WTF::currentThread()), partitionName, static_cast<unsigned>(memoryStats->bucketSlotSize));
 
     WebMemoryAllocatorDump* allocatorDump = m_memoryDump->createMemoryAllocatorDump(dumpName);
     allocatorDump->AddScalar("size", "bytes", memoryStats->residentBytes);