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

Unified Diff: extensions/browser/api/web_request/resource_type_ext.h

Issue 2449913002: Support WebSocket in WebRequest API. (Closed)
Patch Set: Fix unittest; add 'websocket' type to WebRequest API. 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/resource_type_ext.h
diff --git a/extensions/browser/api/web_request/resource_type_ext.h b/extensions/browser/api/web_request/resource_type_ext.h
index 80e5e8e494e663d45db42b0b801a98090d873ec4..5b5811e38b19a8713f43316f2e60a7dd55871f0a 100644
--- a/extensions/browser/api/web_request/resource_type_ext.h
+++ b/extensions/browser/api/web_request/resource_type_ext.h
@@ -5,7 +5,6 @@
#ifndef EXTENSIONS_BROWSER_API_WEB_REQUEST_RESOURCE_TYPE_EXT_H_
#define EXTENSIONS_BROWSER_API_WEB_REQUEST_RESOURCE_TYPE_EXT_H_
-#include "base/logging.h"
#include "content/public/common/resource_type.h"
namespace extensions {
@@ -13,10 +12,7 @@ namespace extensions {
// Extended version of content::ResourceType, which supports a WebSocket type.
struct ResourceTypeExt {
explicit ResourceTypeExt(bool is_websocket = false)
- : is_websocket(is_websocket) {
- DCHECK(!is_websocket);
- // TODO(pkalinnikov): Allow WebSocket type.
- }
+ : is_websocket(is_websocket) {}
explicit ResourceTypeExt(content::ResourceType resource_type)
: resource_type(resource_type) {}

Powered by Google App Engine
This is Rietveld 408576698