Allocator cleanup: allocator_extension call directly into tcmalloc

Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

Also this CL removes a stale base/debug/OWNERS. That file makes
no sense these days as tracing has been moved to base/trace_event/.

BUG=564618
Committed: https://crrev.com/d3a81abc8db50ba646915e3ef700c9ac45958c01
Cr-Commit-Position: refs/heads/master@{#371335}

[Primiano Tucci (use gerrit), 2016-01-20 16:12:11]

Description was changed from

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
This allows to dismantle all the indirection layer in the codebase.
This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.

BUG=564618
==========

to

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

BUG=564618
==========

[Primiano Tucci (use gerrit), 2016-01-20 16:18:18]

primiano@chromium.org changed reviewers:
+ jln@chromium.org, pkasting@chromium.org, thakis@chromium.org

[Primiano Tucci (use gerrit), 2016-01-20 16:18:19]

It's cleanup time

+thakis for base/
+pkasting for chrome/browser/ui
+jln@ for one-line change in content/browser/zygote_host

https://codereview.chromium.org/1607303002/diff/1/base/debug/OWNERS
File base/debug/OWNERS (left):

https://codereview.chromium.org/1607303002/diff/1/base/debug/OWNERS#oldcode1
base/debug/OWNERS:1: per-file trace_event*=nduca@chromium.org
This seems a leftover from the times when tracing was living in base/debug.
Since it's cleanup time...

[Primiano Tucci (use gerrit), 2016-01-20 16:22:28]

+sque/bccheng FYI: this should not have any side effect on CrOs, at least not
intentionally :)

[Peter Kasting, 2016-01-20 20:48:41]

LGTM

[Primiano Tucci (use gerrit), 2016-01-25 18:10:34]

primiano@chromium.org changed reviewers:
+ mdempsky@chromium.org, sievers@chromium.org
- jln@chromium.org

[Primiano Tucci (use gerrit), 2016-01-25 18:10:35]

+sievers for -14 lines in content_main_runner.cc
+mdempsky for 10 lines in zygote_host_impl_linux.cc
PING thakis@ for base/

[mdempsky, 2016-01-25 18:23:00]

On 2016/01/25 18:10:35, Primiano Tucci wrote:
> +mdempsky for 10 lines in zygote_host_impl_linux.cc

LGTM

[no sievers, 2016-01-25 19:18:42]

lgtm

[Primiano Tucci (use gerrit), 2016-01-25 19:33:00]

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

[Primiano Tucci (use gerrit), 2016-01-25 19:33:01]

+thestig for base/

[Will Harris, 2016-01-25 19:49:37]

On 2016/01/25 19:33:01, Primiano Tucci wrote:
> +thestig for base/

base/allocator lgtm

[Nico, 2016-01-25 20:37:57]

lgtm, but please mention the base/debug/OWNERS deletion in the cl description
("while here, remove OWNERS file that has per-file owners for files that have
since moved elsewhere" or similar)

[Primiano Tucci (use gerrit), 2016-01-25 21:42:04]

Description was changed from

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

BUG=564618
==========

to

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

Also this CL removes a stale base/debug/OWNERS. That file makes
no sense these days as tracing has been moved to base/trace_event/.

BUG=564618
==========

[Primiano Tucci (use gerrit), 2016-01-25 21:42:27]

The CQ bit was checked by primiano@chromium.org

[Primiano Tucci (use gerrit), 2016-01-25 21:42:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org
Link to the patchset: https://codereview.chromium.org/1607303002/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2016-01-25 21:43:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1607303002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1607303002/80001

[commit-bot: I haz the power, 2016-01-25 22:21:35]

Description was changed from

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

Also this CL removes a stale base/debug/OWNERS. That file makes
no sense these days as tracing has been moved to base/trace_event/.

BUG=564618
==========

to

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

Also this CL removes a stale base/debug/OWNERS. That file makes
no sense these days as tracing has been moved to base/trace_event/.

BUG=564618
==========

[commit-bot: I haz the power, 2016-01-25 22:21:36]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-01-25 22:22:55]

Description was changed from

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

Also this CL removes a stale base/debug/OWNERS. That file makes
no sense these days as tracing has been moved to base/trace_event/.

BUG=564618
==========

to

==========
Allocator cleanup: allocator_extension call directly into tcmalloc

After crrev.com/1584893002 base/ can use directly allocator.
The grand plan is to dismantle all the indirection layer such that:
 - base is the only thing that knows about allocator-specific details.
 - non-base code uses only base abstractions and stops looking
   at USE_TCMALLOC build flags.

This CL, specifically, addresses the profiling-related methods moving
the tcmalloc-specific dependencies to base and base only.
Other parts of the codebase will be cleaned up in separate CLs.

Also this CL removes a stale base/debug/OWNERS. That file makes
no sense these days as tracing has been moved to base/trace_event/.

BUG=564618

Committed: https://crrev.com/d3a81abc8db50ba646915e3ef700c9ac45958c01
Cr-Commit-Position: refs/heads/master@{#371335}
==========

[commit-bot: I haz the power, 2016-01-25 22:22:56]

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