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

Unified Diff: ext/background.js

Issue 29958567: Issue 7104 - Clear style sheets when frame structure is updated (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Use state object (includes rebase) Created Dec. 18, 2018, 4:32 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 | « no previous file | lib/contentFiltering.js » ('j') | lib/contentFiltering.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ext/background.js
===================================================================
--- a/ext/background.js
+++ b/ext/background.js
@@ -128,16 +128,18 @@
let frame = frames.get(frameId);
if (!frame)
{
frame = {};
frames.set(frameId, frame);
}
+ frame.state = {};
kzar 2018/12/19 10:22:00 Nit: How about `Object.create(null)`?
Manish Jethani 2018/12/19 13:03:54 Done.
+
return frame;
}
function updatePageFrameStructure(frameId, tabId, url, parentFrameId)
{
if (frameId == 0)
{
let page = new Page({id: tabId, url});
« no previous file with comments | « no previous file | lib/contentFiltering.js » ('j') | lib/contentFiltering.js » ('J')

Powered by Google App Engine
This is Rietveld