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

Unified Diff: chrome/ext/content.js

Issue 29371763: Issue 4795 - Use modern JavaScript syntax (Closed)
Patch Set: Fix scoping regression Created Jan. 16, 2017, 9:12 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/content.js
diff --git a/chrome/ext/content.js b/chrome/ext/content.js
index b54fdcfb0c9f568738704a7587554dfce14949d8..6da72ec4e94fe3d58285b6388df0c12ab9ce1680 100644
--- a/chrome/ext/content.js
+++ b/chrome/ext/content.js
@@ -1,14 +1,16 @@
-chrome.runtime.onMessage.addListener(function(message, sender, sendResponse)
+"use strict";
+
+chrome.runtime.onMessage.addListener((message, sender, sendResponse) =>
{
return ext.onMessage._dispatch(message, {}, sendResponse).indexOf(true) != -1;
});
-ext.onExtensionUnloaded = (function()
+ext.onExtensionUnloaded = (() =>
{
- var port = null;
+ let port = null;
return {
- addListener: function(listener)
+ addListener(listener)
{
if (!port)
port = chrome.runtime.connect();
@@ -17,7 +19,7 @@ ext.onExtensionUnloaded = (function()
// background page dies and automatically disconnects all ports
port.onDisconnect.addListener(listener);
},
- removeListener: function(listener)
+ removeListener(listener)
{
if (port)
{

Powered by Google App Engine
This is Rietveld