GitLab

Now that socket_client.h can be moved out of the Blink exposed API
layer, the extra copy step in P2PSocketClientImpl::SendWithPacketId
can also be removed.
This solution was added temporarily in [1], as part of Onion souping
this file.

[1] https://crrev.com/c/1805159/

In summary, this CL:

- moves socket_client.h from blink/public/platform/modules/p2p/
  to blink/renderer/platform/p2p/.
  - adapts the header being moved accordingly.
  - eliminates blink/public/platform/modules/p2p/ altogether.

- P2PSocketClientImpl::SendWithPacketId to operate over WTF::Vector,
  instead of std::vector.

BUG=787254, 919392
R=guidou@chromium.org, haraken@chromium.org

Change-Id: I1c8713cb6c96663cd0d7db8acbac1dfaa4809931
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806979
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697681}