Unified Diff: messageResponder.js
Issue 29544706:
Noissue - Adapt check for devtools panel support for Firefox (Closed)
Patch Set: Rebase
Created Sept. 15, 2017, 4:51 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« background.js ('k')
|
mobile-options.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: messageResponder.js |
| =================================================================== |
| --- a/messageResponder.js |
| +++ b/messageResponder.js |
| @@ -42,6 +42,8 @@ |
| SpecialSubscription |
| } = require("subscriptionClasses"); |
| + const {Services} = Cu.import("resource://gre/modules/Services.jsm"); |
| + |
| // Some modules doesn't exist on Firefox. Moreover, |
| // require() throws an exception on Firefox in that case. |
| // However, try/catch causes the whole function to to be |
| @@ -184,7 +186,9 @@ |
| if (message.what == "features") |
| { |
| return { |
| - devToolsPanel: info.platform == "chromium" |
| + devToolsPanel: info.platform == "chromium" || |
| + info.application == "firefox" && |
| + Services.vc.compare(info.applicationVersion, "54") >= 0 |
| }; |
| } |
« no previous file with comments
|
« background.js ('k')
|
mobile-options.js » ('j')
|
no next file with comments »
