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

Unified Diff: chrome/ext/common.js

Issue 29371763: Issue 4795 - Use modern JavaScript syntax (Closed)
Patch Set: Workaround limiation with iterating element collections in older versions of Chrome Created Jan. 16, 2017, 8:46 a.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
Index: chrome/ext/common.js
diff --git a/chrome/ext/common.js b/chrome/ext/common.js
index 2e5c776dccb493894383117b12bebafc411587b5..127bfb09b4a6f5bb1b9b9ac6604d1a26155d25cd 100644
--- a/chrome/ext/common.js
+++ b/chrome/ext/common.js
@@ -15,8 +15,17 @@
* along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
*/
-(function()
+"use strict";
+
{
+ // Workaround since HTMLCollection and NodeList didn't have iterator support
+ // before Chrome 51.
+ // https://bugs.chromium.org/p/chromium/issues/detail?id=401699
+ if (!(Symbol.iterator in HTMLCollection.prototype))
+ HTMLCollection.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
+ if (!(Symbol.iterator in NodeList.prototype))
+ NodeList.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
+
/* Message passing */
ext.onMessage = new ext._EventTarget();
@@ -26,10 +35,7 @@
ext.backgroundPage = {
sendMessage: chrome.runtime.sendMessage,
- getWindow: function()
- {
- return chrome.extension.getBackgroundPage();
- }
+ getWindow() { return chrome.extension.getBackgroundPage(); }
};
@@ -37,4 +43,4 @@
ext.getURL = chrome.extension.getURL;
ext.i18n = chrome.i18n;
-})();
+}
« no previous file with comments | « chrome/ext/background.js ('k') | chrome/ext/content.js » ('j') | lib/notificationHelper.js » ('J')

Powered by Google App Engine
This is Rietveld