Unified Diff: include.preload.js
Issue 16067002:
Added Safari Support (Closed)
Patch Set: Made description for Safari long again. The 100 char limit given by the extension builder doesn't a…
Created Oct. 31, 2013, 2:12 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« include.postload.js ('k')
|
lib/stats.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: include.preload.js |
| =================================================================== |
| --- a/include.preload.js |
| +++ b/include.preload.js |
| @@ -78,18 +78,26 @@ |
| if (!url) |
| return; |
| - var type = typeMap[tag]; |
| - chrome.extension.sendRequest({reqtype: "should-collapse", url: url, documentUrl: document.URL, type: type}, function(response) |
| - { |
| - if (response && target.parentNode) |
| + ext.backgroundPage.sendMessage( |
| { |
| - // <frame> cannot be removed, doing that will mess up the frameset |
| - if (tag == "frame") |
| - target.style.setProperty("visibility", "hidden", "!important"); |
| - else |
| - target.parentNode.removeChild(target); |
| + 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 |
| + if (tag == "frame") |
| + target.style.setProperty("visibility", "hidden", "!important"); |
| + else |
| + target.parentNode.removeChild(target); |
| + } |
| } |
| - }); |
| + ); |
| } |
| } |
| @@ -102,10 +110,18 @@ |
| document.addEventListener("error", checkCollapse, true); |
| document.addEventListener("load", checkCollapse, true); |
| - chrome.extension.sendRequest({reqtype: "get-settings", selectors: true, frameUrl: window.location.href}, function(response) |
| - { |
| - setElemhideCSSRules(response.selectors); |
| - }); |
| + ext.backgroundPage.sendMessage( |
| + { |
| + type: "get-settings", |
| + selectors: true, |
| + frameUrl: window.location.href |
| + }, |
| + |
| + function(response) |
| + { |
| + setElemhideCSSRules(response.selectors); |
| + } |
| + ); |
| } |
| // In Chrome 18 the document might not be initialized yet |
« no previous file with comments
|
« include.postload.js ('k')
|
lib/stats.js » ('j')
|
no next file with comments »
