Don't collect results in Menu.setApplicationMenu

This commit is contained in:
Kevin Sawicki 2016-03-10 13:26:14 -08:00
parent 3a1e837f8b
commit 93939089ec

View file

@ -267,7 +267,6 @@ Menu.prototype._callMenuWillShow = function() {
var applicationMenu = null;
Menu.setApplicationMenu = function(menu) {
var j, len, results, w, windows;
if (!(menu === null || menu.constructor === Menu)) {
throw new TypeError('Invalid menu');
}
@ -279,15 +278,11 @@ Menu.setApplicationMenu = function(menu) {
return;
}
menu._callMenuWillShow();
return bindings.setApplicationMenu(menu);
bindings.setApplicationMenu(menu);
} else {
windows = BrowserWindow.getAllWindows();
results = [];
for (j = 0, len = windows.length; j < len; j++) {
w = windows[j];
results.push(w.setMenu(menu));
}
return results;
BrowserWindow.getAllWindows().forEach(function(window) {
window.setMenu(menu);
});
}
};