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

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

Issue 2449913002: Support WebSocket in WebRequest API. (Closed)
Patch Set: Extract process and frame ID from WS requests. Created 4 years, 1 month 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 c4ac79580cec811af010da24ee14d79d50018e26..fd8aaec01dacd515b1c6dcb25e866b888d376135 100644
--- a/extensions/browser/api/web_request/web_request_permissions.cc
+++ b/extensions/browser/api/web_request/web_request_permissions.cc
@@ -75,9 +75,8 @@ bool IsSensitiveURL(const GURL& url) {
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());
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698