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

Unified Diff: ext/background.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.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ext/background.js
diff --git a/ext/background.js b/ext/background.js
index bb935066d52b662225d45000313205ec6cbbce90..723758e1522d2858636e21ab627c2a754a78cdad 100644
--- a/ext/background.js
+++ b/ext/background.js
@@ -96,9 +96,12 @@
return frame.url;
}
},
- sendMessage(message, responseCallback)
+ sendMessage(message, responseCallback, frameId)
{
- chrome.tabs.sendMessage(this.id, message, responseCallback);
+ let options = {};
+ if (typeof frameId != "undefined")
+ options.frameId = frameId;
+ chrome.tabs.sendMessage(this.id, message, options, responseCallback);
}
};
@@ -449,7 +452,7 @@
contexts: item.contexts,
onclick(info, tab)
{
- item.onclick(new Page(tab));
+ item.onclick(new Page(tab), info);
}
});
});
« composer.postload.js ('K') | « composer.postload.js ('k') | lib/filterComposer.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld