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

Unified Diff: lib/messaging.js

Issue 29374674: Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Patch Set: Rebased, restored adblockplusui dependency revision Created March 22, 2017, 7:10 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
« dependencies ('K') | « lib/io.js ('k') | lib/notificationHelper.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/messaging.js
diff --git a/lib/messaging.js b/lib/messaging.js
index fb57704cf345de3590223bdf0ff9a3e9620ce585..7231ee7098cbb43c6037a204f5747a34a86aa356 100644
--- a/lib/messaging.js
+++ b/lib/messaging.js
@@ -31,7 +31,7 @@ function Port()
this._eventEmitter = new EventEmitter();
this._onMessage = this._onMessage.bind(this);
ext.onMessage.addListener(this._onMessage);
-};
+}
Port.prototype = {
_onMessage(message, sender, sendResponse)
@@ -47,7 +47,8 @@ Port.prototype = {
{
response.then(
sendResponse,
- reason => {
+ reason =>
+ {
console.error(reason);
sendResponse(undefined);
}
@@ -55,9 +56,7 @@ Port.prototype = {
async = true;
}
else if (typeof response != "undefined")
- {
sendResponse(response);
- }
}
return async;
« dependencies ('K') | « lib/io.js ('k') | lib/notificationHelper.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld