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

Delta Between Two Patch Sets: devtools-panel.js

Issue 5646124035604480: Issue 154 - Added UI for devtools panel on Chrome (Closed)
Left Patch Set: Addressed Sven's comments and use an HTML template for table rows Created Jan. 13, 2015, 12:16 p.m.
Right Patch Set: Added new request types Created Jan. 28, 2016, 7:37 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « devtools-panel.html ('k') | ext/background.js » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 /* 1 /*
2 * This file is part of Adblock Plus <http://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2015 Eyeo GmbH 3 * Copyright (C) 2006-2016 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 * GNU General Public License for more details. 12 * GNU General Public License for more details.
13 * 13 *
14 * You should have received a copy of the GNU General Public License 14 * You should have received a copy of the GNU General Public License
15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. 15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
16 */ 16 */
17
18 "use strict";
17 19
18 function generateFilter(request) 20 function generateFilter(request)
19 { 21 {
20 var filter = request.url.replace(/^[\w\-]+:\/+(?:www\.)?/, "||"); 22 var filter = request.url.replace(/^[\w\-]+:\/+(?:www\.)?/, "||");
21 23
22 if (request.type == "POPUP") 24 if (request.type == "POPUP")
23 { 25 {
24 filter += "$popup"; 26 filter += "$popup";
25 27
26 if (request.url == "about:blank") 28 if (request.url == "about:blank")
27 filter += ",domain=" + request.docDomain; 29 filter += ",domain=" + request.docDomain;
28 } 30 }
29 31
30 return filter; 32 return filter;
31 } 33 }
32 34
33 function createActionButton(action, label, filter, port) 35 function createActionButton(action, label, filter)
34 { 36 {
35 var button = document.createElement("span"); 37 var button = document.createElement("span");
36 38
37 button.textContent = label; 39 button.textContent = label;
38 button.classList.add("action"); 40 button.classList.add("action");
39 41
40 button.addEventListener("click", function() 42 button.addEventListener("click", function()
41 { 43 {
42 port.postMessage({action: action, filter: filter}); 44 ext.backgroundPage.sendMessage({
43 }); 45 type: "filters." + action,
46 text: filter
47 });
48 }, false);
44 49
45 return button; 50 return button;
46 } 51 }
47 52
48 function createRecord(request, filter, port, template) 53 function createRecord(request, filter, template)
49 { 54 {
50 var row = document.importNode(template, true); 55 var row = document.importNode(template, true);
56 row.dataset.type = request.type;
51 57
52 row.querySelector(".domain").textContent = request.docDomain; 58 row.querySelector(".domain").textContent = request.docDomain;
53 row.querySelector(".type" ).textContent = request.type; 59 row.querySelector(".type").textContent = request.type;
54 60
55 var urlElement = row.querySelector(".url"); 61 var urlElement = row.querySelector(".url");
56 var actionWrapper = row.querySelector(".action-wrapper"); 62 var actionWrapper = row.querySelector(".action-wrapper");
57 63
58 if (request.url) 64 if (request.url)
59 { 65 {
60 urlElement.textContent = request.url; 66 urlElement.textContent = request.url;
61 67
62 if ("openResource" in ext.devtools.panels && request.type != "POPUP") 68 if ("openResource" in ext.devtools.panels && request.type != "POPUP")
63 { 69 {
64 urlElement.classList.add("resourceLink"); 70 urlElement.classList.add("resourceLink");
65 urlElement.addEventListener("click", function() 71 urlElement.addEventListener("click", function()
66 { 72 {
67 ext.devtools.panels.openResource(request.url); 73 ext.devtools.panels.openResource(request.url);
68 }); 74 }, false);
69 } 75 }
70 } 76 }
71 77
72 if (filter) 78 if (filter)
73 { 79 {
74 var filterElement = row.querySelector(".filter"); 80 var filterElement = row.querySelector(".filter");
75 var originElement = row.querySelector(".origin"); 81 var originElement = row.querySelector(".origin");
76 82
77 filterElement.textContent = filter.text; 83 filterElement.textContent = filter.text;
78 row.dataset.state = filter.whitelisted ? "whitelisted" : "blocked"; 84 row.dataset.state = filter.whitelisted ? "whitelisted" : "blocked";
79 85
80 if (filter.subscription) 86 if (filter.subscription)
81 originElement.textContent = filter.subscription; 87 originElement.textContent = filter.subscription;
82 else 88 else
83 { 89 {
84 if (filter.userDefined) 90 if (filter.userDefined)
85 originElement.textContent = "user-defined"; 91 originElement.textContent = "user-defined";
86 else 92 else
87 originElement.textContent = "unnamed subscription"; 93 originElement.textContent = "unnamed subscription";
88 94
89 originElement.classList.add("unnamed"); 95 originElement.classList.add("unnamed");
90 } 96 }
91 97
92 if (!filter.whitelisted && request.type != "ELEMHIDE") 98 if (!filter.whitelisted && request.type != "ELEMHIDE")
93 actionWrapper.appendChild(createActionButton("add", "Add exception", "@@" + generateFilter(request), port)); 99 actionWrapper.appendChild(createActionButton("add", "Add exception", "@@" + generateFilter(request)));
94 100
95 if (filter.userDefined) 101 if (filter.userDefined)
96 actionWrapper.appendChild(createActionButton("remove", "Remove rule", filt er.text, port)); 102 actionWrapper.appendChild(createActionButton("remove", "Remove rule", filt er.text));
97 } 103 }
98 else 104 else
99 actionWrapper.appendChild(createActionButton("add", "Block item", generateFi lter(request), port)); 105 actionWrapper.appendChild(createActionButton("add", "Block item", generateFi lter(request)));
100 106
101 return row; 107 return row;
102 } 108 }
103 109
104 document.addEventListener("DOMContentLoaded", function() 110 document.addEventListener("DOMContentLoaded", function()
105 { 111 {
106 var port = ext.devtools.connect();
107
108 var container = document.getElementById("items"); 112 var container = document.getElementById("items");
109 var table = container.querySelector("tbody"); 113 var table = container.querySelector("tbody");
110 var template = document.querySelector("template").content.firstElementChild; 114 var template = document.querySelector("template").content.firstElementChild;
111 115
112 port.onMessage.addListener(function(msg) 116 document.getElementById("reload").addEventListener("click", function()
113 { 117 {
114 switch (msg.type) 118 ext.devtools.inspectedWindow.reload();
119 }, false);
120
121 document.getElementById("filter-state").addEventListener("change", function(ev ent)
122 {
123 container.dataset.filterState = event.target.value;
124 }, false);
125
126 document.getElementById("filter-type").addEventListener("change", function(eve nt)
127 {
128 container.dataset.filterType = event.target.value;
129 }, false);
130
131 ext.onMessage.addListener(function(message)
132 {
133 switch (message.type)
115 { 134 {
116 case "add-record": 135 case "add-record":
117 table.appendChild(createRecord(msg.request, msg.filter, port, template)) ; 136 table.appendChild(createRecord(message.request, message.filter, template ));
118 break; 137 break;
119 138
120 case "update-record": 139 case "update-record":
121 var oldRow = table.getElementsByTagName("tr")[msg.index]; 140 var oldRow = table.getElementsByTagName("tr")[message.index];
122 var newRow = createRecord(msg.request, msg.filter, port, template); 141 var newRow = createRecord(message.request, message.filter, template);
123 oldRow.parentNode.replaceChild(newRow, oldRow); 142 oldRow.parentNode.replaceChild(newRow, oldRow);
124 newRow.classList.add("changed"); 143 newRow.classList.add("changed");
125 container.classList.add("has-changes"); 144 container.classList.add("has-changes");
126 break; 145 break;
127 146
128 case "remove-record": 147 case "remove-record":
129 var row = table.getElementsByTagName("tr")[msg.index]; 148 var row = table.getElementsByTagName("tr")[message.index];
130 row.parentNode.removeChild(row); 149 row.parentNode.removeChild(row);
131 container.classList.add("has-changes"); 150 container.classList.add("has-changes");
132 break; 151 break;
133 152
134 case "reset": 153 case "reset":
135 table.innerHTML = ""; 154 table.innerHTML = "";
136 container.classList.remove("has-changes"); 155 container.classList.remove("has-changes");
137 break; 156 break;
138 } 157 }
139 }); 158 });
140 159 }, false);
141 document.getElementById("reload").addEventListener("click", function()
142 {
143 ext.devtools.inspectedWindow.reload();
144 });
145
146 document.getElementById("filter-state").addEventListener("change", function(ev ent)
147 {
148 container.dataset.filterState = event.target.value;
149 });
150
151 document.getElementById("filter-type").addEventListener("change", function(eve nt)
152 {
153 container.dataset.filterType = event.target.value;
154 });
155 });
LEFTRIGHT

Powered by Google App Engine
This is Rietveld