[tracing] Simplify MemoryDumpProvider interface

[tracing] Simplify MemoryDumpProvider interface

- Rename DumpInto(PMD*) into OnMemoryDump(PMD*)
- Make the MemoryDumpProvider interface pure abstract.
- Move the task_runner argument to MDM.RegisterDumpProvider().
- Remove the awkward DeclareAttribute() logic
- Rename the useless MemoryDumpProvider.GetFriendlyName() method.
- Adapt the existing allocators dump providers to the changes.

BUG=466141
Committed: https://crrev.com/2d56df1b08b8c8439315efc3f6af2dc053db16a0
Cr-Commit-Position: refs/heads/master@{#327257}

[Primiano Tucci (use gerrit), 2015-04-27 22:40:28]

primiano@chromium.org changed reviewers:
+ nduca@chromium.org

[Primiano Tucci (use gerrit), 2015-04-27 22:40:28]

PTAL, this is just removing files

This is part of a series of 3 CLs to cleanup the MDM/MAD interface in
preparation for the upcoming Blink plumbing.

1*) crrev.com/1107093004 [THIS CL] (Simplify MemoryDumpProvider interface)
2) crrev.com/1104413002 (Get rid of MemoryAllocatorAttributesTypeInfo)  
3) crrev.com/1105143003 (Simplify the MemoryAllocatorDump API)

[reveman, 2015-04-27 23:01:23]

reveman@chromium.org changed reviewers:
+ reveman@chromium.org

[reveman, 2015-04-27 23:01:24]

https://codereview.chromium.org/1107093004/diff/1/base/trace_event/memory_dum...
File base/trace_event/memory_dump_provider.h (left):

https://codereview.chromium.org/1107093004/diff/1/base/trace_event/memory_dum...
base/trace_event/memory_dump_provider.h:50: virtual ~MemoryDumpProvider();
Can we keep an empty protected dtor to prevent this from being destroyed
incorrectly?

[nduca, 2015-04-28 00:25:54]

lgtm

[Primiano Tucci (use gerrit), 2015-04-28 07:55:54]

Patchset #2 (id:20001) has been deleted

[Primiano Tucci (use gerrit), 2015-04-28 08:03:03]

https://codereview.chromium.org/1107093004/diff/1/base/trace_event/memory_dum...
File base/trace_event/memory_dump_provider.h (left):

https://codereview.chromium.org/1107093004/diff/1/base/trace_event/memory_dum...
base/trace_event/memory_dump_provider.h:50: virtual ~MemoryDumpProvider();
On 2015/04/27 23:01:24, reveman wrote:
> Can we keep an empty protected dtor to prevent this from being destroyed
> incorrectly?

Done.

[Primiano Tucci (use gerrit), 2015-04-28 08:03:13]

The CQ bit was checked by primiano@chromium.org

[Primiano Tucci (use gerrit), 2015-04-28 08:03:13]

The patchset sent to the CQ was uploaded after l-g-t-m from nduca@chromium.org
Link to the patchset: https://codereview.chromium.org/1107093004/#ps40001
(title: "Keep protected dtor")

[commit-bot: I haz the power, 2015-04-28 08:03:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1107093004/40001

[commit-bot: I haz the power, 2015-04-28 10:06:33]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-04-28 10:07:31]

Patchset 2 (id:??) landed as
https://crrev.com/2d56df1b08b8c8439315efc3f6af2dc053db16a0
Cr-Commit-Position: refs/heads/master@{#327257}