DescriptionRevert of macOS: Fix a bug in logic to get all memory regions. (patchset #1 id:1 of https://codereview.chromium.org/2786733004/ )
Reason for revert:
This failed ProcessMetricsMemoryDumpProviderTest.NoDuplicateRegions on "Mac ASan 64 Tests (1)".
https://findit-for-me.appspot.com/waterfall/failure?url=https://build.chromium.org/p/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/28499
Original issue's description:
> macOS: Fix a bug in logic to get all memory regions.
>
> The logic was accidentally double-counting the size of a memory region, causing
> some memory regions to get skipped.
>
> BUG=706547
>
> Review-Url: https://codereview.chromium.org/2786733004
> Cr-Commit-Position: refs/heads/master@{#460585}
> Committed: https://chromium.googlesource.com/chromium/src/+/f955ebf64b34ec9cbdcb137c00bc9bba1867d071
TBR=primiano@chromium.org,erikchen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=706547
Review-Url: https://codereview.chromium.org/2788453002
Cr-Commit-Position: refs/heads/master@{#460619}
Committed: https://chromium.googlesource.com/chromium/src/+/6387395d1a02579a33652086a64384311d576ba0
Patch Set 1 #
Messages
Total messages: 7 (4 generated)
|