commit 10c9e9ed129660b720fc9ba336a6f45dad7c0ec4
author Primiano Tucci <primiano@google.com> Fri Jan 08 13:04:40 2021 +0100
committer Primiano Tucci <primiano@google.com> Fri Jan 08 13:04:40 2021 +0100
tree 026af5c0fd2288ba66eb0f81f43d4520f76ff873
parent 3ce4e8ee1301d0102b3595187a93a3ba067b2a69

Minor changes to IPC layer in preparation for Win port.

This CL doesn't introduce any major change (neither on
Windows nor on Linux). It mainly refactor the codebase
to make the Windows IPC cls easier to review:
- Rename the GN target platform_posix to platform_impl
  (Soon the same target will have a platform_windows.cc)
- Add a platform_unittest to cover the platform TLS logic.
- Refactor ThreadTaskRunner to directly expose a TaskRunner
  interface (soon platform_windows.cc will need it as well).
- Refactor a bunch of places to take a ScopedSocketHandle
  rather than a ScopedFile. They are the same type on POSIX
  but not on Windows (where the latter wraps a SOCKET).
- Remove the fd() argument from the base SharedMemory class.
  That makes sense only on POSIX and it caused a lot of empty
  implementation around (note: this will cause a Chromium
  roll failure, we need to match the removal in .MojoSharedMemory)
- Minor other tweaks to get rid of platform-specific includes.

Bug: 174454879
Test: manual on Windows.
Change-Id: Iead1ec87213c56fd3bb170c72d482f71fa33e7e3