Unified Diff: lib/filterComposer.js
Issue 29370947:
Issue 3138 - Improve how context menu "block element" handles iframes (Closed)
Patch Set: Rebased
Created Aug. 9, 2017, 3:27 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« composer.postload.js ('K')
|
« ext/background.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterComposer.js |
| diff --git a/lib/filterComposer.js b/lib/filterComposer.js |
| index 8da9e97d5aa66faabf090030aab09d05aefd40d9..960a0cd3196c7f58b929f3de678a750756feb5f2 100644 |
| --- a/lib/filterComposer.js |
| +++ b/lib/filterComposer.js |
| @@ -168,9 +168,11 @@ function composeFilters(details) |
| let contextMenuItem = { |
| title: ext.i18n.getMessage("block_element"), |
| contexts: ["image", "video", "audio"], |
| - onclick(page) |
| + onclick(page, info) |
| { |
| - page.sendMessage({type: "composer.content.contextMenuClicked"}); |
| + page.sendMessage( |
| + {type: "composer.content.contextMenuClicked"}, undefined, info.frameId |
| + ); |
| } |
| }; |
« composer.postload.js ('K')
|
« ext/background.js ('k')
|
no next file »
|
no next file with comments »
