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

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

Issue 2700553002: Introduce WebRequestResourceType. (Closed)
Patch Set: Address comments from Devlin. 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_resource_type.cc
diff --git a/extensions/browser/api/web_request/web_request_resource_type.cc b/extensions/browser/api/web_request/web_request_resource_type.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6882ff59b887b072aa5df36687be521d8a8b88fe
--- /dev/null
+++ b/extensions/browser/api/web_request/web_request_resource_type.cc
@@ -0,0 +1,23 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "extensions/browser/api/web_request/web_request_resource_type.h"
+
+#include "base/logging.h"
+#include "content/public/browser/resource_request_info.h"
+#include "net/url_request/url_request.h"
+
+namespace extensions {
+
+WebRequestResourceType ToWebRequestResourceType(
+ const net::URLRequest* request) {
+ DCHECK(request);
+ if (request->url().SchemeIsWSOrWSS())
+ return WebRequestResourceType::WEB_SOCKET;
+ const auto* info = content::ResourceRequestInfo::ForRequest(request);
+ return info ? ToWebRequestResourceType(info->GetResourceType())
+ : WebRequestResourceType::OTHER;
Devlin 2017/02/16 16:08:26 and then return INVALID if info is null rather tha
pkalinnikov 2017/02/16 19:27:33 Done.
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698