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

Unified Diff: include.preload.js

Issue 16067002: Added Safari Support (Closed)
Patch Set: Created Oct. 21, 2013, 8:11 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
« chrome/background.js ('K') | « include.postload.js ('k') | lib/utils.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include.preload.js
===================================================================
--- a/include.preload.js
+++ b/include.preload.js
@@ -78,9 +78,12 @@
if (!url)
return;
- var type = typeMap[tag];
- chrome.extension.sendRequest({reqtype: "should-collapse", url: url, documentUrl: document.URL, type: type}, function(response)
- {
+ ext.backgroundPage.sendMessage({
+ type: "should-collapse",
+ url: url,
+ documentUrl: document.URL,
+ mediatype: typeMap[tag]
+ }, function(response) {
if (response && target.parentNode)
{
// <frame> cannot be removed, doing that will mess up the frameset
@@ -102,8 +105,11 @@
document.addEventListener("error", checkCollapse, true);
document.addEventListener("load", checkCollapse, true);
- chrome.extension.sendRequest({reqtype: "get-settings", selectors: true, frameUrl: window.location.href}, function(response)
- {
+ ext.backgroundPage.sendMessage({
+ type: "get-settings",
+ selectors: true,
+ frameUrl: window.location.href
+ }, function(response) {
setElemhideCSSRules(response.selectors);
});
}
« chrome/background.js ('K') | « include.postload.js ('k') | lib/utils.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld