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

Side by Side Diff: chrome/content/frameScript.js

Issue 29323439: Issue 2877 - Element Hiding Helper causes tab to crash when selecting some elements (Closed)
Patch Set: Created Aug. 10, 2015, 9:22 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | lib/aardvark.js » ('j') | lib/aardvark.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
saroyanm 2015/08/11 08:41:50 Did this file exist ? I can't find it -> https://
Wladimir Palant 2015/08/11 10:05:18 Wrong repository, https://hg.adblockplus.org/elemh
saroyanm 2015/08/11 13:50:14 Ah right :)
2 * This Source Code is subject to the terms of the Mozilla Public License 2 * This Source Code is subject to the terms of the Mozilla Public License
3 * version 2.0 (the "License"). You can obtain a copy of the License at 3 * version 2.0 (the "License"). You can obtain a copy of the License at
4 * http://mozilla.org/MPL/2.0/. 4 * http://mozilla.org/MPL/2.0/.
5 */ 5 */
6 6
7 (function() 7 (function()
8 { 8 {
9 const Cu = Components.utils; 9 const Cu = Components.utils;
10 10
11 let rand = Components.stack.filename.replace(/.*\?/, ""); 11 let rand = Components.stack.filename.replace(/.*\?/, "");
12 let module = "chrome://elemhidehelper/content/actor.jsm?" + rand; 12 let module = "chrome://elemhidehelper/content/actor.jsm?" + rand;
13 let {shutdown, getNodeInfo, togglePreview, forgetNode} = Cu.import(module, {}) ; 13 let {shutdown, getNodeInfo, togglePreview, forgetNode} = Cu.import(module, {}) ;
14 14
15 addMessageListener("ElemHideHelper:Shutdown", onShutdown); 15 addMessageListener("ElemHideHelper:Shutdown", onShutdown);
16 addMessageListener("ElemHideHelper:GetNodeInfo", onGetNodeInfo); 16 addMessageListener("ElemHideHelper:GetNodeInfo", onGetNodeInfo);
17 addMessageListener("ElemHideHelper:Preview", onTogglePreview); 17 addMessageListener("ElemHideHelper:Preview", onTogglePreview);
18 18
19 function onShutdown() 19 function onShutdown()
20 { 20 {
21 shutdown(); 21 shutdown();
22 Cu.unload(module); 22 Cu.unload(module);
23 removeMessageListener("ElemHideHelper:Shutdown", onShutdown); 23 removeMessageListener("ElemHideHelper:Shutdown", onShutdown);
24 removeMessageListener("ElemHideHelper:GetNodeInfo", onGetNodeInfo); 24 removeMessageListener("ElemHideHelper:GetNodeInfo", onGetNodeInfo);
25 removeMessageListener("ElemHideHelper:Preview", onTogglePreview); 25 removeMessageListener("ElemHideHelper:Preview", onTogglePreview);
26 } 26 }
27 27
28 function onGetNodeInfo(message) 28 function onGetNodeInfo(message)
29 { 29 {
30 let nodeInfo = getNodeInfo(message.objects.element); 30 let nodeInfo = getNodeInfo(message.objects.element);
31 message.objects.callback(JSON.stringify(nodeInfo)); 31 nodeInfo.messageId = message.data;
32 sendAsyncMessage("ElemHideHelper:GetNodeInfo:Response", nodeInfo);
32 } 33 }
33 34
34 function onTogglePreview(message) 35 function onTogglePreview(message)
35 { 36 {
36 togglePreview(message.data.nodeID, message.data.stylesheetData); 37 togglePreview(message.data.nodeID, message.data.stylesheetData);
37 if (message.data.forgetNode) 38 if (message.data.forgetNode)
38 forgetNode(message.data.nodeID); 39 forgetNode(message.data.nodeID);
39 } 40 }
40 })(); 41 })();
OLDNEW
« no previous file with comments | « no previous file | lib/aardvark.js » ('j') | lib/aardvark.js » ('J')

Powered by Google App Engine
This is Rietveld