[tracing] Adding Skia memory dump provider with cache totals

skia/ext/skia_memory_dump_provider.h

+// 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.
+
+#ifndef SKIA_EXT_SKIA_MEMORY_DUMP_PROVIDER_H_
+#define SKIA_EXT_SKIA_MEMORY_DUMP_PROVIDER_H_
+
+#include "SkPreConfig.h"

[reed1, 2015/07/27 19:11:47]

Hmmm, why do you need this? In general, we think PreConfig and PostConfig are
sort of "private" headers to skia.

[ssid, 2015/07/27 19:17:21]

This is needed for the SK_API define. Is that not required?

[reed1, 2015/07/27 20:13:48]

#include "SkTypes.h" is our "base" header. You should include that if you just
need access to something like SK_API.

+#include "base/memory/singleton.h"
+#include "base/trace_event/memory_dump_provider.h"
+
+namespace skia {
+
+class SK_API SkiaMemoryDumpProvider
+    : public base::trace_event::MemoryDumpProvider {
+ public:
+  static SkiaMemoryDumpProvider* GetInstance();
+
+  // base::trace_event::MemoryDumpProvider implementation:
+  bool OnMemoryDump(
+      base::trace_event::ProcessMemoryDump* process_memory_dump) override;
+
+ private:
+  friend struct DefaultSingletonTraits<SkiaMemoryDumpProvider>;
+
+  SkiaMemoryDumpProvider();
+  ~SkiaMemoryDumpProvider() override;
+
+  DISALLOW_COPY_AND_ASSIGN(SkiaMemoryDumpProvider);
+};
+
+}  // namespace skia
+
+#endif  // SKIA_EXT_SKIA_MEMORY_DUMP_PROVIDER_H_