Introduce interface for memory dumps

tests/TraceMemoryDumpTest.cpp

+/*
+ * Copyright 2015 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#include "SkTraceMemoryDump.h"
+
+#include "SkString.h"
+#include "Test.h"
+
+/*
+ * Build test for SkTraceMemoryDump.
+ */
+SkTraceMemoryDump::~SkTraceMemoryDump() {}

[ericrk, 2015/08/19 12:45:28]

This is a bit odd? shouldn't be needed if you inline the destructor - which we
seem to do in other cases.

[Primiano Tucci (use gerrit), 2015/08/20 08:48:01]

Done.

+
+class TestSkTraceMemoryDump : public SkTraceMemoryDump {
+public:
+  TestSkTraceMemoryDump() {}
+  virtual ~TestSkTraceMemoryDump() {}

[ericrk, 2015/08/19 12:45:28]

override not virtual?

[Primiano Tucci (use gerrit), 2015/08/20 08:48:01]

Done.

+
+  void DumpScalarValue(const SkString &dumpName, const char *name,
+                       const char *units, uint64_t value) override {}
+
+  void SetMemoryBacking(const SkString &dumpName, const SkString &backingType,
+                        const SkString &backingObjectId) override{};
+  void SetDiscardableMemoryBacking(
+      const SkString &dumpName,
+      const SkDiscardableMemory &discardableMemoryObject) override {}
+};
+
+DEF_TEST(SkTraceMemoryDump, reporter) {
+  TestSkTraceMemoryDump x;
+  x.DumpScalarValue(SkString("foo"), "size", "bytes", 42);
+}