Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(410)

Unified Diff: extensions/browser/api/web_request/web_request_permissions.cc

Issue 2449913002: Support WebSocket in WebRequest API. (Closed)
Patch Set: git cl format Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/browser/api/web_request/web_request_permissions.cc
diff --git a/extensions/browser/api/web_request/web_request_permissions.cc b/extensions/browser/api/web_request/web_request_permissions.cc
index 11e8c9d1054ae421e7a14d0b209c8610959e574f..8b2f2f49742858767d6cbab0465b34f81dd181df 100644
--- a/extensions/browser/api/web_request/web_request_permissions.cc
+++ b/extensions/browser/api/web_request/web_request_permissions.cc
@@ -31,9 +31,8 @@ namespace {
bool HasWebRequestScheme(const GURL& url) {
return (url.SchemeIs(url::kAboutScheme) || url.SchemeIs(url::kFileScheme) ||
url.SchemeIs(url::kFileSystemScheme) ||
- url.SchemeIs(url::kFtpScheme) || url.SchemeIs(url::kHttpScheme) ||
- url.SchemeIs(url::kHttpsScheme) ||
- url.SchemeIs(extensions::kExtensionScheme));
+ url.SchemeIs(url::kFtpScheme) || url.SchemeIsHTTPOrHTTPS() ||
+ url.SchemeIs(extensions::kExtensionScheme) || url.SchemeIsWSOrWSS());
}
bool g_allow_all_extension_locations_in_public_session = false;
« no previous file with comments | « extensions/browser/api/web_request/web_request_event_details.cc ('k') | extensions/common/api/web_request.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698