GitLab

Get rid of the SlotMetadata constructor/destructor and replace the
destructor with a Destroy() method. Also change alloc_ptr to be a
uintptr_t. This refactor makes SlotMetadata safer to initialize with
out-of-process memory in an upcoming crash handler change, because the
SlotMetadata we read should not be automatically destructed (the members
will be from another memory space). Since alloc_ptr is only used in the
crash handler there's no reason to not make it a uintptr_t to make it
appear less like a pointer in crash handler's memory space.

Bug: 896019
Change-Id: I3f1f22eac6277722de9594a3331b8daef618d29d
Reviewed-on: https://chromium-review.googlesource.com/c/1340868
Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>
Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608935}