Revert 183089

Revert 183089

Needs owner review

> Linux: add option to use system tcmalloc (off by default)
> 
> This changes things only for Linux distros, Google Chrome
> will continue to use bundled tcmalloc.
> 
> TBR=darin
> 
> BUG=174634
> 
> Review URL: https://codereview.chromium.org/12224030

TBR=phajdan.jr@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=183104

[Paweł Hajdan Jr., 2013-02-18 12:00:21]

LGTM

I'm sorry this made you upset. My TBR was based on no change for Google Chrome,
and no response from reviewers after individual pings. I'll make sure they
respond now and I'll address any remaining concerns.

[jschuh, 2013-02-18 13:04:52]

I'm not upset, but I am very concerned at how the original CL was landed. If
you're going to unbundle a library you need to first consult with owners of that
library to ensure it's a reasonable course of action. In this case, I'm almost
certain it's not (for the reasons I provided in the original CL.

More broadly, I don't think there's ever justification for using TBR on such an
impactful change as unbundling a library. Yes, it doesn't affect the Chrome
Google ships, but it does negatively affect the Chromium project if it causes
Linux distributions and embedders to ship a grossly inferior browser.

[jschuh, 2013-02-18 13:15:32]

Also, to be clear, I'm no stranger to TBR. Particularly since I started the
Win64 port, where I use it many times a week. However, I try very hard to ensure
I use it only for trivial changes that don't impact anything other than Chrome's
Win64 configuration. For anything else I get a review (accepting a few minor
slip ups on my part).