Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/csp.js

Issue 29705719: Issue 6402 - Split filter hit / request logging out into own API (Closed)
Patch Set: Nightmarish rebase Created April 13, 2018, 3:20 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « include.preload.js ('k') | lib/cssInjection.js » ('j') | lib/devtools.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/csp.js
diff --git a/lib/csp.js b/lib/csp.js
index 1e9a3670ee8dddb31fea270bb105615a7b881bc4..b8664b915a630b6fb28ac99292fda99df2e07cd3 100644
--- a/lib/csp.js
+++ b/lib/csp.js
@@ -24,7 +24,7 @@ const {extractHostFromFrame, getDecodedHostname,
isThirdParty, stringifyURL} = require("./url");
const {checkWhitelisted} = require("./whitelisting");
const {FilterNotifier} = require("filterNotifier");
-const devtools = require("./devtools");
+const {logRequest} = require("./hitLogger");
const {typeMap} = RegExpFilter;
@@ -60,8 +60,8 @@ browser.webRequest.onHeadersReceived.addListener(details =>
return;
}
- devtools.logRequest([details.tabId], urlString, "CSP", hostname,
- thirdParty, null, specificOnly, cspMatch);
+ logRequest([details.tabId], urlString, "CSP", hostname, thirdParty, null,
+ specificOnly, cspMatch);
FilterNotifier.emit("filter.hitCount", cspMatch, 0, 0, [details.tabId]);
if (cspMatch instanceof WhitelistFilter)
« no previous file with comments | « include.preload.js ('k') | lib/cssInjection.js » ('j') | lib/devtools.js » ('J')

Powered by Google App Engine
This is Rietveld