New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: migrate protocol module to NetworkService (Part 7) #18290
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, the netlog failures is not related.
@@ -17,6 +17,10 @@ namespace api { | |||
|
|||
namespace { | |||
|
|||
const char* kBuiltinSchemes[] = { | |||
"about", "file", "http", "https", "data", "filesystem", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe using values from url_constants
is better, but this is going to be short lived code. Since we plan to remove it once the service is enabled, I am okay with leaving this as it is.
No Release Notes |
Description of Change
Refs #15791.
This PR makes the
registerFileProtocol
API supporting reading asar files when NetworkService is enabled. The tests ofregisterFileProtocol
are passing now.Checklist
npm test
passesRelease Notes
Notes: no-notes