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
34 files changed