Support WebSocket in WebRequest API.

extensions/common/url_pattern.cc

Index: extensions/common/url_pattern.cc
diff --git a/extensions/common/url_pattern.cc b/extensions/common/url_pattern.cc
index fcd50f676dcf709777b3df33894adeecd3b213e9..af7ed375febfbcd433c1b9b45dc8a0910b1cae36 100644
--- a/extensions/common/url_pattern.cc
+++ b/extensions/common/url_pattern.cc
@@ -31,7 +31,8 @@ const char* const kValidSchemes[] = {
     url::kHttpScheme,         url::kHttpsScheme,
     url::kFileScheme,         url::kFtpScheme,
     content::kChromeUIScheme, extensions::kExtensionScheme,
-    url::kFileSystemScheme,
+    url::kFileSystemScheme,   url::kWsScheme,
+    url::kWssScheme,
 };
 
 const int kValidSchemeMasks[] = {
@@ -42,6 +43,8 @@ const int kValidSchemeMasks[] = {
   URLPattern::SCHEME_CHROMEUI,
   URLPattern::SCHEME_EXTENSION,
   URLPattern::SCHEME_FILESYSTEM,
+  URLPattern::SCHEME_WS,
+  URLPattern::SCHEME_WSS,
 };
 
 static_assert(arraysize(kValidSchemes) == arraysize(kValidSchemeMasks),
@@ -319,7 +322,7 @@ bool URLPattern::SetScheme(base::StringPiece scheme) {
   spec_.clear();
   scheme.CopyToString(&scheme_);
   if (scheme_ == "*") {
-    valid_schemes_ &= (SCHEME_HTTP | SCHEME_HTTPS);
+    valid_schemes_ &= (SCHEME_HTTP | SCHEME_HTTPS | SCHEME_WS | SCHEME_WSS);

[Devlin, 2017/02/14 01:21:38]

Hmm... I'm not sure this is preferable.  We don't want most APIs to be able to
(try to) do anything with ws:/wss: schemes.  I'd be inclined to say we should
have them specify them explicitly separately, as we do for other "unusual"
schemes.

[pkalinnikov, 2017/02/14 13:49:51]

SGTM.

   } else if (!IsValidScheme(scheme_)) {
     return false;
   }