Improve test coveage of WebProcessMemoryDumpImplTest

third_party/WebKit/Source/platform/web_process_memory_dump_impl_test.cc

 // 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.
 
 #include "platform/web_process_memory_dump_impl.h"
 
+#include "base/memory/discardable_memory.h"
+#include "base/test/test_discardable_memory_allocator.h"
 #include "base/trace_event/memory_allocator_dump.h"
 #include "base/trace_event/process_memory_dump.h"
 #include "base/trace_event/trace_event_argument.h"
 #include "base/values.h"
 #include "platform/web_memory_allocator_dump_impl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "wtf/OwnPtr.h"
 
 namespace blink {
 
@@ skipping 95 matching lines @@
 
   // Check that AddOwnershipEdge is propagated correctly.
   auto wmad4 = wpmd1->createMemoryAllocatorDump("1/4");
   wpmd1->addOwnershipEdge(wmad4->guid(), guid);
   auto allocator_dumps_edges =
       wpmd1->process_memory_dump()->allocator_dumps_edges();
   ASSERT_EQ(1u, allocator_dumps_edges.size());
   ASSERT_EQ(wmad4->guid(), allocator_dumps_edges[0].source.ToUint64());
   ASSERT_EQ(guid, allocator_dumps_edges[0].target.ToUint64());
 
+  // Check that createDumpAdapterForSkia() works.
+  auto skia_trace_memory_dump = wpmd1->createDumpAdapterForSkia("1/skia");
+  ASSERT_NE(nullptr, skia_trace_memory_dump);

[Primiano Tucci (use gerrit), 2016/02/29 16:15:28]

IIRC the ASSERT_NE incantation fails to expand properly this and will fail to
compile on some platforms (I hope I am wrong and it got fixed).
IIRC either you ASSERT_NE(static_cast<SkiaBlaBla*>(nullptr), ...) (or
equivalently cast the 2nd to void*)
or more simply ASSERT_TRUE(skia_trace_memory_dump) 

[hajimehoshi, 2016/03/01 10:06:23]

Done. (Used ASSERT_TRUE)

+
+  // Check that createDiscardableMemoryAllocatorDump() works.
+  base::TestDiscardableMemoryAllocator discardable_memory_allocator;
+  auto discardable_memory =
+      discardable_memory_allocator.AllocateLockedDiscardableMemory(1024);
+  wpmd1->createDiscardableMemoryAllocatorDump("1/discardable",
+                                              discardable_memory.get());
+  discardable_memory->Unlock();
+
   wpmd1.reset();
 }
 
 }  // namespace blink