tree fa970204ab38ac69d58192f0dbab4aa25ac18dd2
parent 66ccf4be6575a97dc277844eb602886ac54146c9
author Wez <wez@chromium.org> 1511816951 +0900
committer Qijiang Fan <fqj@google.com> 1591311149 +0900

Revert "Migrate call-sites away from un-managed PlatformHandle[Vector]s."

This reverts commit 7df6feaddfff789abbef65a84c1a4296c5927a0a.

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/791390
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519398}

CrOS-Libchrome-Original-Commit: 2c820264b8fbb36234c19b5e3a7e4849a4b30471