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

Side by Side Diff: ext/background.js

Issue 29371763: Issue 4795 - Use modern JavaScript syntax (Closed)
Patch Set: Created Jan. 13, 2017, 12:11 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
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2016 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 17
18 (function()
19 { 18 {
20 var nonEmptyPageMaps = Object.create(null); 19 let nonEmptyPageMaps = Object.create(null);
21 var pageMapCounter = 0; 20 let pageMapCounter = 0;
22 21
23 var PageMap = ext.PageMap = function() 22 let PageMap = ext.PageMap = function()
24 { 23 {
25 this._map = Object.create(null); 24 this._map = Object.create(null);
26 this._id = ++pageMapCounter; 25 this._id = ++pageMapCounter;
27 }; 26 };
28 PageMap.prototype = { 27 PageMap.prototype = {
29 _delete: function(id) 28 _delete: function(id)
30 { 29 {
31 delete this._map[id]; 30 delete this._map[id];
32 31
33 if (Object.keys(this._map).length == 0) 32 if (Object.keys(this._map).length == 0)
(...skipping 11 matching lines...) Expand all
45 { 44 {
46 this._map[page.id] = value; 45 this._map[page.id] = value;
47 nonEmptyPageMaps[this._id] = this; 46 nonEmptyPageMaps[this._id] = this;
48 }, 47 },
49 has: function(page) 48 has: function(page)
50 { 49 {
51 return page.id in this._map; 50 return page.id in this._map;
52 }, 51 },
53 clear: function() 52 clear: function()
54 { 53 {
55 for (var id in this._map) 54 for (let id in this._map)
56 this._delete(id); 55 this._delete(id);
57 }, 56 },
58 delete: function(page) 57 delete: function(page)
59 { 58 {
60 this._delete(page.id); 59 this._delete(page.id);
61 } 60 }
62 }; 61 };
63 62
64 ext._removeFromAllPageMaps = function(pageId) 63 ext._removeFromAllPageMaps = pageId =>
65 { 64 {
66 for (var pageMapId in nonEmptyPageMaps) 65 for (let pageMapId in nonEmptyPageMaps)
67 nonEmptyPageMaps[pageMapId]._delete(pageId); 66 nonEmptyPageMaps[pageMapId]._delete(pageId);
68 }; 67 };
69 })(); 68 }
OLDNEW
« chrome/ext/common.js ('K') | « composer.postload.js ('k') | ext/common.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld