commit a3e53de8e9031d13b3c4e32d3fc65207d4200ee9
author primiano <primiano@chromium.org> Mon Dec 07 08:51:04 2015 -0800
committer Commit bot <commit-bot@chromium.org> Mon Dec 07 16:51:54 2015 +0000
tree 490a8f5045fc158b161c4f2fb9a9d72e55136476
parent b7cba6e88c7fb798af4150913756656bbe32f82a

Remove allocator_extension_thunks since this layer is not required

Original patch author: ssid@chromium.org

Original CL description:
crrev.com/10239012 added thunks target. There allocator_shim.cc wanted
to set function pointers when heap_init_() was called. To do this,
allocator_shim.cc (part of allocator target) used thunks to store the
functions without actually depending on thunks (relying on the fact
that base would finally link thunks). Since allocator_shim.cc included
thunks.h and allocator_unittests included allocator, but not base,
thunks target was introduced to avoid depending on base. Current
situation is that thunks is no longer used by the shim layer and not
used by allocator_unittests either (after shim was removed). Thunks was
left around for no reason. See design doc in the bug.

This CL removes the target and stores the functions in base.

BUG=564618
TBR=wfh@chromium.org,thakis@chromium.org
TBR Reason=the original patch was reviewed in crrev.com/1487403002.
           Just rebasing and landing it as ssid is OOO for one week.

Review URL: https://codereview.chromium.org/1505743002

Cr-Original-Commit-Position: refs/heads/master@{#363497}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: dda6c278fb665a69a6c4ef2d08f2a02b7f588f6b