GitLab

This reverts commit 7df6fead.

Reason for revert: Likely responsible for crbug.com/788192 and crbug.com/788559 - crashes and aborts under Mac & Linux (so likely a bug in POSIX-specific changes).

Original change's description:
> Migrate call-sites away from un-managed PlatformHandle[Vector]s.
> 
> As the first step to merging ScopedPlatformHandle into PlatformHandle
> to have a single, always-managed container for platform handles, we:
> 
> - Replace ScopedPlatformHandleVectorPtr and PlatformHandleVector
>   with vector<ScopedPlatformHandle>.
> - Replace PlatformHandle with ScopedPlatformHandle in some core APIs.
> - Use const& to pass handles to APIs without passing ownership, for
>   both individual platform handles, and vectors.
> 
> Bug: 753541
> Change-Id: Id965dcf3770ef11bf36f78277fa7199ea779599e
> Reviewed-on: https://chromium-review.googlesource.com/742309
> Reviewed-by: Ricky Liang <jcliang@chromium.org>
> Reviewed-by: Ken Rockot <rockot@chromium.org>
> Reviewed-by: Dominick Ng <dominickn@chromium.org>
> Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
> Commit-Queue: Wez <wez@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#518343}

TBR=posciak@chromium.org,wez@chromium.org,jcliang@chromium.org,rockot@chromium.org,lhchavez@chromium.org,elijahtaylor@chromium.org,dominickn@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 753541
Change-Id: I890e833ae6d68ac4a346d230781ca0ac4798f135
Reviewed-on: https://chromium-review.googlesource.com/791390Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519398}