Revert of Move logic from memory_mac to allocator_interception_mac.

Revert of Move logic from memory_mac to allocator_interception_mac. (patchset #5 id:80001 of https://codereview.chromium.org/2649973003/ )

Reason for revert:
This seems to be causing compile to fail on the Mac GPU FYI Asan bot, with error:
../../base/allocator/allocator_interception_mac.mm:320:3: error: use of undeclared identifier 'DeprotectMallocZone'
  DeprotectMallocZone(zone, &reprotection_start, &reprotection_length,
  ^
1 error generated.

See https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20GPU%20ASAN%20Release/builds/1407/steps/compile/logs/stdio
and
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20GPU%20ASAN%20Release?numbuilds=100

Original issue's description:
> Move logic from memory_mac to allocator_interception_mac.
> 
> This CL is a refactor and has no intended behavior changes.
> 
> This CL moves the allocator interception logic from base/process/memory_mac.mm
> to base/allocator/allocator_interception_mac.mm in preparation for an
> implementation of the allocator shim.
> 
> BUG=665567
> 
> Review-Url: https://codereview.chromium.org/2649973003
> Cr-Commit-Position: refs/heads/master@{#446234}
> Committed: https://chromium.googlesource.com/chromium/src/+/9550b71347ce9acbe2875fb75ea46ec096230506

TBR=primiano@chromium.org,mark@chromium.org,erikchen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=665567
Review-Url: https://codereview.chromium.org/2658083002
Cr-Commit-Position: refs/heads/master@{#446346}
Committed: https://chromium.googlesource.com/chromium/src/+/e9942e23fec9e2bca8ebd70f79feb2120943b54b

[ajuma, 2017-01-26 16:33:58]

The CQ bit was checked by ajuma@chromium.org

[ajuma, 2017-01-26 16:33:58]

Created Revert of Move logic from memory_mac to allocator_interception_mac.

[commit-bot: I haz the power, 2017-01-26 16:34:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-26 16:35:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485448438357610, "parent_rev":
"cb3a30a6acb4e3e774f9c0d416bb90c270065364", "commit_rev":
"e9942e23fec9e2bca8ebd70f79feb2120943b54b"}

[commit-bot: I haz the power, 2017-01-26 16:36:05]

Description was changed from

==========
Revert of Move logic from memory_mac to allocator_interception_mac. (patchset #5
id:80001 of https://codereview.chromium.org/2649973003/ )

Reason for revert:
This seems to be causing compile to fail on the Mac GPU FYI Asan bot, with
error:
../../base/allocator/allocator_interception_mac.mm:320:3: error: use of
undeclared identifier 'DeprotectMallocZone'
  DeprotectMallocZone(zone, &reprotection_start, &reprotection_length,
  ^
1 error generated.

See
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20GPU%20ASAN%20Rel...
and
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20GPU%20ASAN%20Rel...

Original issue's description:
> Move logic from memory_mac to allocator_interception_mac.
> 
> This CL is a refactor and has no intended behavior changes.
> 
> This CL moves the allocator interception logic from base/process/memory_mac.mm
> to base/allocator/allocator_interception_mac.mm in preparation for an
> implementation of the allocator shim.
> 
> BUG=665567
> 
> Review-Url: https://codereview.chromium.org/2649973003
> Cr-Commit-Position: refs/heads/master@{#446234}
> Committed:
https://chromium.googlesource.com/chromium/src/+/9550b71347ce9acbe2875fb75ea4...

TBR=primiano@chromium.org,mark@chromium.org,erikchen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=665567
==========

to

==========
Revert of Move logic from memory_mac to allocator_interception_mac. (patchset #5
id:80001 of https://codereview.chromium.org/2649973003/ )

Reason for revert:
This seems to be causing compile to fail on the Mac GPU FYI Asan bot, with
error:
../../base/allocator/allocator_interception_mac.mm:320:3: error: use of
undeclared identifier 'DeprotectMallocZone'
  DeprotectMallocZone(zone, &reprotection_start, &reprotection_length,
  ^
1 error generated.

See
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20GPU%20ASAN%20Rel...
and
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20GPU%20ASAN%20Rel...

Original issue's description:
> Move logic from memory_mac to allocator_interception_mac.
> 
> This CL is a refactor and has no intended behavior changes.
> 
> This CL moves the allocator interception logic from base/process/memory_mac.mm
> to base/allocator/allocator_interception_mac.mm in preparation for an
> implementation of the allocator shim.
> 
> BUG=665567
> 
> Review-Url: https://codereview.chromium.org/2649973003
> Cr-Commit-Position: refs/heads/master@{#446234}
> Committed:
https://chromium.googlesource.com/chromium/src/+/9550b71347ce9acbe2875fb75ea4...

TBR=primiano@chromium.org,mark@chromium.org,erikchen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=665567

Review-Url: https://codereview.chromium.org/2658083002
Cr-Commit-Position: refs/heads/master@{#446346}
Committed:
https://chromium.googlesource.com/chromium/src/+/e9942e23fec9e2bca8ebd70f79fe...
==========

[commit-bot: I haz the power, 2017-01-26 16:36:06]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e9942e23fec9e2bca8ebd70f79fe...

[erikchen, 2017-01-26 19:00:41]

On 2017/01/26 16:36:06, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/src/+/e9942e23fec9e2bca8ebd70f79fe...

Please note that this reverted the wrong CL. You actually wanted:
https://codereview.chromium.org/2650363002/

[ajuma, 2017-01-26 19:07:06]

On 2017/01/26 19:00:41, erikchen wrote:
> On 2017/01/26 16:36:06, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1) as
> >
>
https://chromium.googlesource.com/chromium/src/+/e9942e23fec9e2bca8ebd70f79fe...
> 
> Please note that this reverted the wrong CL. You actually wanted:
> https://codereview.chromium.org/2650363002/

Yeah, sorry about that! (Also note that this had the effect of reverting both
CLs.)

[Primiano Tucci (use gerrit), 2017-01-26 19:10:32]

On 2017/01/26 19:07:06, ajuma wrote:
> On 2017/01/26 19:00:41, erikchen wrote:
> > On 2017/01/26 16:36:06, commit-bot: I haz the power wrote:
> > > Committed patchset #1 (id:1) as
> > >
> >
>
https://chromium.googlesource.com/chromium/src/+/e9942e23fec9e2bca8ebd70f79fe...
> > 
> > Please note that this reverted the wrong CL. You actually wanted:
> > https://codereview.chromium.org/2650363002/
> 
> Yeah, sorry about that! (Also note that this had the effect of reverting both
> CLs.)

Yeah that sounds a genuine problem with the revert button. Erik filed
crbug.com/685715. No worries.