[tracing] Expose memory-infra tracing infrastructure to Blink.

[tracing] Expose memory-infra tracing infrastructure to Blink.

Exposes the core memory-infra classes to Blink. These classes are, in
the very essence, a reproposition of base::trace_event ones and keep
the same semantic.
The Chromium-side changes are in crrev.com/1108403006
For more documentation see the design doc: http://goo.gl/Bdyql6

BUG=480500
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194822

[Primiano Tucci (use gerrit), 2015-04-29 22:11:58]

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

[Primiano Tucci (use gerrit), 2015-04-29 22:11:58]

Hello! Here it is all the dumper plumbing as agreed.
Design doc  with example here: http://goo.gl/Bdyql6

[haraken, 2015-04-30 01:45:49]

The API looks good in terms of the memory-infra design (I'm not an API owner).

Where are you going to add tests?

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryAl...
File public/platform/WebMemoryAllocatorDump.h (right):

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryAl...
public/platform/WebMemoryAllocatorDump.h:20: //   name: name of the attribute
(e.g., "outer_size", "inner_size", "objects_count").

Wrap the comment into 80 characters. The same comment for other comments.

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryDu...
File public/platform/WebMemoryDumpProvider.h (right):

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryDu...
public/platform/WebMemoryDumpProvider.h:19: // Called by the memory dump manager
when generating memory dumps. Embedders are expected to

memory dump manager => MemoryDumpManager

[Primiano Tucci (use gerrit), 2015-04-30 09:07:42]

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

[Primiano Tucci (use gerrit), 2015-04-30 09:07:43]

+jochen as /public OWNER

Fixed comments. 
I am working on the tests right now, but I think is worth reducing CR latency
and getting this reviewed in the meanwhile.

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryAl...
File public/platform/WebMemoryAllocatorDump.h (right):

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryAl...
public/platform/WebMemoryAllocatorDump.h:20: //   name: name of the attribute
(e.g., "outer_size", "inner_size", "objects_count").
On 2015/04/30 01:45:49, haraken wrote:
> 
> Wrap the comment into 80 characters. The same comment for other comments.
> 

Done.

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryDu...
File public/platform/WebMemoryDumpProvider.h (right):

https://codereview.chromium.org/1111383002/diff/1/public/platform/WebMemoryDu...
public/platform/WebMemoryDumpProvider.h:19: // Called by the memory dump manager
when generating memory dumps. Embedders are expected to
On 2015/04/30 01:45:49, haraken wrote:
> 
> memory dump manager => MemoryDumpManager

Done.

[Primiano Tucci (use gerrit), 2015-04-30 16:57:02]

primiano@chromium.org changed reviewers:
+ pdr@chromium.org
- jochen@chromium.org

[Primiano Tucci (use gerrit), 2015-04-30 16:57:02]

+pdr, could you PTAL to this? This is to expose go/memory-infra to blink and
enable dumping of memory stats from blink subsystems.

-jochen (Friday is a holiday in DE, and Monday is in UK... bad timing)

[dsinclair, 2015-05-01 16:53:10]

dsinclair@chromium.org changed reviewers:
+ dsinclair@chromium.org, pdr@chromium.orgl
- pdr@chromium.org

[dsinclair, 2015-05-01 16:53:11]

lgtm

[dsinclair, 2015-05-01 16:53:23]

dsinclair@chromium.org changed reviewers:
+ pdr@chromium.org
- pdr@chromium.orgl

[Rick Byers, 2015-05-01 17:09:28]

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

[Rick Byers, 2015-05-01 17:09:28]

public/ RS LGTM (given the design review that's already occurred here, on the
doc, and in the chromium side implementation of these APIs).

[Primiano Tucci (use gerrit), 2015-05-01 17:12:32]

On 2015/04/30 01:45:49, haraken wrote:
> The API looks good in terms of the memory-infra design (I'm not an API owner).
> 
> Where are you going to add tests?

For the discussion about the test see crrev.com/1108403006.
The only place where I can see this being tested is on the chromium side (but
there it is just plumbing against base/).
Will take a look to that anyways on the beginning of next week.

[Primiano Tucci (use gerrit), 2015-05-01 17:12:40]

The CQ bit was checked by primiano@chromium.org

[commit-bot: I haz the power, 2015-05-01 17:12:57]

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

[commit-bot: I haz the power, 2015-05-01 18:45:12]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194822