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

Unified Diff: content/shell/test_runner/mock_web_document_subresource_filter.cc

Issue 2724903004: Add ELEMENT_TYPE_WEBSOCKET to subresource filter (Closed)
Patch Set: rebase Created 3 years, 9 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: content/shell/test_runner/mock_web_document_subresource_filter.cc
diff --git a/content/shell/test_runner/mock_web_document_subresource_filter.cc b/content/shell/test_runner/mock_web_document_subresource_filter.cc
index 3502258d2e687d8ddcbfbb67ad4420fdbaef1dd9..6cd2d56993abcb9921ab97bebf3e3d1bb8c587b4 100644
--- a/content/shell/test_runner/mock_web_document_subresource_filter.cc
+++ b/content/shell/test_runner/mock_web_document_subresource_filter.cc
@@ -21,12 +21,23 @@ MockWebDocumentSubresourceFilter::~MockWebDocumentSubresourceFilter() {}
blink::WebDocumentSubresourceFilter::LoadPolicy
MockWebDocumentSubresourceFilter::getLoadPolicy(
const blink::WebURL& resource_url,
- blink::WebURLRequest::RequestContext /* ignored */) {
- const std::string resource_path(GURL(resource_url).path());
+ blink::WebURLRequest::RequestContext) {
+ return getLoadPolicyImpl(resource_url);
+}
+
+blink::WebDocumentSubresourceFilter::LoadPolicy
+MockWebDocumentSubresourceFilter::getLoadPolicyForWebSocketConnect(
+ const blink::WebURL& url) {
+ return getLoadPolicyImpl(url);
+}
+
+blink::WebDocumentSubresourceFilter::LoadPolicy
+MockWebDocumentSubresourceFilter::getLoadPolicyImpl(const blink::WebURL& url) {
+ const std::string path(GURL(url).path());
return std::find_if(disallowed_path_suffixes_.begin(),
disallowed_path_suffixes_.end(),
- [&resource_path](const std::string& suffix) {
- return base::EndsWith(resource_path, suffix,
+ [&path](const std::string& suffix) {
+ return base::EndsWith(path, suffix,
base::CompareCase::SENSITIVE);
}) == disallowed_path_suffixes_.end()
? Allow

Powered by Google App Engine
This is Rietveld 408576698