New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 816634 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Jul 16
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug

Blocking:
issue 721401
issue 844146



Sign in to add a comment

Switch content/browser/renderer_host/p2p/socket_host_udp.h to using NetworkService's UDP socket API

Project Member Reported by xunji...@chromium.org, Feb 26 2018

Issue description

content/browser/renderer_host/p2p/socket_host_udp.h uses net::DatagramServerSocket. As a part of network servicification, all interactions with //net needs to be done through mojo APIs.

Network Service has a mojo UDP socket API (see //services/network/public/mojom/udp_socket.mojom). content/browser/renderer_host/p2p/socket_host_udp.h needs to be migrated to using it.

 
Examples of migrating to udp_socket.mojom can be found in
extensions/browser/api/socket/socket_api.h  &
media/cast/net/udp_socket_client.h


WebRTC folks, can someone take this?

Comment 2 by mef@chromium.org, Mar 12 2018

An example of migrating to udp_socket.mojom can be found in https://chromium-review.googlesource.com/c/chromium/src/+/904266. 
 

Comment 3 by mef@chromium.org, Mar 12 2018

Owner: mef@chromium.org
Status: Assigned (was: Available)
I
Owner: steveanton@chromium.org
steveanton@ started looking into this. 

Comment 5 by dxie@chromium.org, May 15 2018

Labels: -Pri-3 Proj-Servicification-Canary OS-All Pri-1
Blocking: 844146

Comment 7 by dxie@chromium.org, May 18 2018

Labels: -OS-All OS-Windows OS-Linux OS-Mac OS-Chrome OS-Android
Owner: ----
Status: Available (was: Assigned)
Owner: xunji...@chromium.org
Status: Assigned (was: Available)
Status: WontFix (was: Assigned)
Decided to adopt Approach 2 in https://docs.google.com/document/d/1PeRCLWgJdiwznysVOGyXJCF7tDsMQgwto1x_6v3dHug/edit?usp=sharing instead.

Closing this one as WontFix.

Sign in to add a comment