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

Unified Diff: background.js

Issue 29573905: Issue 4580 - Replace ext.devtools with devtools Base URL: https://hg.adblockplus.org/adblockplusui/
Patch Set: Remove devtools.onCreated Created Oct. 11, 2017, 11:34 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 | devtools-panel.js » ('j') | ext/content.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -536,18 +536,22 @@
whitelisted: isWhitelisted
}
]
}
}, "*");
}, 1000);
}
- ext.devtools.onCreated.addListener((panel) =>
+ let {port} = require("messaging");
+
+ port.on("devtools.ready", (message, sender) =>
Manish Jethani 2017/10/11 23:37:33 There's no API called devtools.onCreated. This is
{
+ let panel = sender.page;
+
// blocked request
panel.sendMessage({
type: "add-record",
request: {
url: "http://adserver.example.com/ad_banner.png",
type: "IMAGE",
docDomain: "example.com"
},
« no previous file with comments | « no previous file | devtools-panel.js » ('j') | ext/content.js » ('J')

Powered by Google App Engine
This is Rietveld