tcmalloc: fixed config_linux.h to build for arm targets.

tcmalloc: fixed config_linux.h to build for arm targets.

BUG=chromium-os:34620
TEST=trybot


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158903

[asharif1, 2012-09-24 23:29:08]

Please review this fix.

[asharif1, 2012-09-25 01:54:03]

On 2012/09/24 23:29:08, asharif1 wrote:
> Please review this fix.

+reviewer: gpike.

[gpike, 2012-09-25 05:41:41]

lgtm

Sounds like the right fix for upstream is to modify configure.ac. For now this
seems OK though.

[Alexander Potapenko, 2012-09-25 09:40:03]

We don't use configure.ac in Chromium, so a proper fix will require committing a
patch to configure.ac upstream and then creating config_linux_arm.h in Chromium
(by either copying and patching config_linux.h or re-generating the config from
the upstream source) and adding it to allocator.gyp.
I'm fine with this CL for now, but it would be better with a small TODO comment
that can be noticed by the reviewers during the next tcmalloc merge.

[asharif1, 2012-09-25 17:45:02]

On 2012/09/25 09:40:03, Alexander Potapenko wrote:
> We don't use configure.ac in Chromium, so a proper fix will require committing
a
> patch to configure.ac upstream and then creating config_linux_arm.h in
Chromium
> (by either copying and patching config_linux.h or re-generating the config
from
> the upstream source) and adding it to allocator.gyp.
> I'm fine with this CL for now, but it would be better with a small TODO
comment
> that can be noticed by the reviewers during the next tcmalloc merge.

Please take a look again.

[Alexander Potapenko, 2012-09-26 09:08:15]

LGTM

[commit-bot: I haz the power, 2012-09-26 18:36:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asharif@chromium.org/10970030/3002

[commit-bot: I haz the power, 2012-09-26 20:44:50]

Retried try job too often for step(s) browser_tests

[commit-bot: I haz the power, 2012-09-26 20:57:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asharif@chromium.org/10970030/3002

[commit-bot: I haz the power, 2012-09-26 21:47:21]

Change committed as 158903