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

Unified Diff: lib/contentFiltering.js

Issue 29894564: Issue 6999 - Remove inlineEmulated flag in content.applyFilters (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Rebase Created Sept. 28, 2018, 7:39 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
« include.preload.js ('K') | « include.preload.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/contentFiltering.js
===================================================================
--- a/lib/contentFiltering.js
+++ b/lib/contentFiltering.js
@@ -243,23 +243,16 @@
{
inline = true;
}
let response = {trace, inline, emulatedPatterns};
if (trace || inline)
response.selectors = selectors;
- // If we can't remove user style sheets using tabs.removeCSS, we'll only keep
- // adding them, which could cause problems with emulation filters as
- // described in issue #5864. Instead, we can just ask the content script to
- // add styles for emulation filters inline.
- if (!styleSheetRemovalSupported)
- response.inlineEmulated = true;
-
return response;
});
port.on("content.injectSelectors", (message, sender) =>
{
updateFrameStyles(sender.page.id, sender.frame.id, message.selectors,
message.groupName, message.appendOnly);
});
« include.preload.js ('K') | « include.preload.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld