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

Unified Diff: chrome/devtools.js

Issue 29338491: Issue 3823 - Split up message responder code (Closed)
Patch Set: Moved get-domain-enabled-state to whitelisting module Created March 22, 2016, 8:22 a.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 | « background.js ('k') | include.preload.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/devtools.js
===================================================================
--- a/chrome/devtools.js
+++ b/chrome/devtools.js
@@ -17,10 +17,16 @@
"use strict";
-chrome.runtime.sendMessage({type: "has-devtools-panel"}, function(response)
-{
- if (response)
- chrome.devtools.panels.create("Adblock Plus",
- "icons/detailed/abp-48.png",
- "devtools-panel.html");
-});
+chrome.runtime.sendMessage(
+ {
+ type: "prefs.get",
+ key: "show_devtools_panel"
+ },
+ function(enabled)
+ {
+ if (enabled)
+ chrome.devtools.panels.create("Adblock Plus",
+ "icons/detailed/abp-48.png",
+ "devtools-panel.html");
+ }
+);
« no previous file with comments | « background.js ('k') | include.preload.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld