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

Unified Diff: lib/cssInjection.js

Issue 29705719: Issue 6402 - Split filter hit / request logging out into own API (Closed)
Patch Set: Fixed typo and exposed pageId for logged requests Created Feb. 22, 2018, 3:55 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/devLogger.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/cssInjection.js
diff --git a/lib/cssInjection.js b/lib/cssInjection.js
index 30977ed1248e33c74a1ef20ed05693a02eb36eef..9e2d32851d5326613cba13ccd9904c9d1d78c94d 100644
--- a/lib/cssInjection.js
+++ b/lib/cssInjection.js
@@ -25,7 +25,7 @@ const {ElemHideEmulation} = require("elemHideEmulation");
const {checkWhitelisted} = require("whitelisting");
const {extractHostFromFrame} = require("url");
const {port} = require("messaging");
-const devtools = require("devtools");
+const {DevLogger} = require("devLogger");
const info = require("info");
// Chromium's support for tabs.removeCSS is still a work in progress and the
@@ -138,7 +138,7 @@ port.on("elemhide.getSelectors", (message, sender) =>
{
let selectors = [];
let emulatedPatterns = [];
- let trace = devtools && devtools.hasPanel(sender.page);
+ let trace = DevLogger.listening(sender.page.id);
let inline = !userStyleSheetsSupported;
if (!checkWhitelisted(sender.page, sender.frame,
« no previous file with comments | « include.preload.js ('k') | lib/devLogger.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld