[tracing] directly use memory-infra from blink -> base

[tracing] directly use memory-infra from blink -> base

Now that the blink repo has been merged there isn't any reason to keep
the memory-infra glue layer around.
This CL makes direct use of the few /base/trace_event/ headers for
memory-infra from blink:
 - base/trace_event/memory_allocator_dump.h
 - base/trace_event/memory_dump_provider.h
 - base/trace_event/process_memory_dump.h

A follow-up CL (crrev.com/1394183003) will take care of removing the useless
glue layer classes.
This CL depends on crrev.com/1382583002 for the wtf::String
toUTF8StdString() method.

BUG=541262

[Primiano Tucci (use gerrit), 2015-09-28 16:05:57]

Patchset #1 (id:1) has been deleted

[esprehn, 2015-09-28 18:03:13]

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

[esprehn, 2015-09-28 18:03:14]

typo in your patch title? :P

https://codereview.chromium.org/1375643002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.cpp
(right):

https://codereview.chromium.org/1375643002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.cpp:28:
return std::string(reinterpret_cast<const char*>(str.ascii().data()));
This cast doesn't make sense, data() is already a "const char*"

[Primiano Tucci (use gerrit), 2015-09-28 19:07:40]

https://codereview.chromium.org/1375643002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.cpp
(right):

https://codereview.chromium.org/1375643002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.cpp:28:
return std::string(reinterpret_cast<const char*>(str.ascii().data()));
On 2015/09/28 18:03:14, esprehn wrote:
> This cast doesn't make sense, data() is already a "const char*"

Yeah, just a leftover of doing character8() which returns unsigned char* before
somebody (you?) pointed out that it is not null terminated.

[Primiano Tucci (use gerrit), 2015-10-08 18:08:08]

Patchset #2 (id:40001) has been deleted

[Primiano Tucci (use gerrit), 2015-10-08 18:16:26]

primiano@chromium.org changed reviewers:
+ haraken@chromium.org, ssid@chromium.org
- esprehn@chromium.org

[Primiano Tucci (use gerrit), 2015-10-08 18:16:26]

Ok, there is a lot of cleanup we can do here.
The actual deletion happens in crrev.com/1394183003 which gets rid 
-700 lines \o/

Now this is really blocked on the toUTF8StdString() CL (Crrev.com/1382583002)

[Ruud van Asseldonk, 2015-10-08 18:40:55]

https://codereview.chromium.org/1375643002/diff/60001/base/trace_event/memory...
File base/trace_event/memory_dump_provider.h (right):

https://codereview.chromium.org/1375643002/diff/60001/base/trace_event/memory...
base/trace_event/memory_dump_provider.h:36: static void Register(
Does this introduce two ways of doing the same thing,
|MemoryDumpProvider::Register(mdp)| vs
|MemoryDumpManager::GetInstance()->RegisterDumpProvider(mdp)|? Why is that
required?

[Primiano Tucci (use gerrit), 2015-10-09 09:25:19]

Given https://codereview.chromium.org/1382583002/ let's put this on hold until
blink owners don't figure out a plan.
We'll keep using the glue layer as usual until then.

On 2015/10/08 18:40:55, Ruud van Asseldonk wrote:
>
https://codereview.chromium.org/1375643002/diff/60001/base/trace_event/memory...
> File base/trace_event/memory_dump_provider.h (right):
> 
>
https://codereview.chromium.org/1375643002/diff/60001/base/trace_event/memory...
> base/trace_event/memory_dump_provider.h:36: static void Register(
> Does this introduce two ways of doing the same thing,
> |MemoryDumpProvider::Register(mdp)| vs
> |MemoryDumpManager::GetInstance()->RegisterDumpProvider(mdp)|? Why is that
> required?
This was to avoid including memory_dump_manager.h. Specifically its tracing
includes end up defining the same macros that are defined in the blink fork of
tracing.
All this will be solved once we properly get rid of all the tracing layers in
blink.

In the meantime, keep plumbing as usual

[Primiano Tucci (use gerrit), 2015-10-09 09:25:30]

primiano@chromium.org changed reviewers:
- haraken@chromium.org, ssid@chromium.org