Expose direct socket APIs via [DirectSocketEnabled]. #28861
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch introduces a new placeholder IDL extended attribute for the
set of restrictions which might enable us to safely expose direct
sockets, as well as a set of other APIs that have similar
outside-the-web threat models. It then applies that IDL extended
attribute to the existing direct socket IDL files.
For the moment, this imposes two requirements on Direct Sockets:
I expect these requirements to shift in the future, but this is a
reasonable place to start.
Patch 5/5:
Note that this CL removes the Direct Socket interfaces from the
webexposed
test suite. That's a little too complicated to fix inthis CL, but I aim to fix it in https://crbug.com/1206656.
Bug: 1206150
Change-Id: I519e1c5d164c43f5870c45823d08063f4a3af682
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2874890
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: Eric Willigers <ericwilligers@chromium.org>
Commit-Queue: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#880498}