Unified Diff: chrome/ext/popup.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.
« chrome/ext/common.js ('K')
|
« chrome/ext/devtools.js ('k')
|
composer.js » ('j')
|
ext/background.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: chrome/ext/popup.js |
| diff --git a/chrome/ext/popup.js b/chrome/ext/popup.js |
| index 40932374211505989553c15352d2e7215e7d07da..2a892cfe24db52286b0d6d384625d0bc6e5cb6ea 100644 |
| --- a/chrome/ext/popup.js |
| +++ b/chrome/ext/popup.js |
| @@ -1,9 +1,10 @@ |
| -(function() |
| +"use strict"; |
| + |
| { |
| - var backgroundPage = chrome.extension.getBackgroundPage(); |
| + let backgroundPage = chrome.extension.getBackgroundPage(); |
| window.ext = Object.create(backgroundPage.ext); |
| - ext.closePopup = function() |
| + ext.closePopup = () => |
| { |
| window.close(); |
| }; |
| @@ -13,9 +14,6 @@ |
| ext.backgroundPage = { |
| sendMessage: chrome.runtime.sendMessage, |
| - getWindow: function() |
| - { |
| - return backgroundPage; |
| - } |
| + getWindow() { return backgroundPage; } |
| }; |
| -})(); |
| +} |
« chrome/ext/common.js ('K')
|
« chrome/ext/devtools.js ('k')
|
composer.js » ('j')
|
ext/background.js » ('J')
