From 842ee5ca3c532e188c5aa647ae133ee884f01f84 Mon Sep 17 00:00:00 2001 From: Feross Aboukhadijeh Date: Fri, 19 Aug 2016 20:04:18 -0700 Subject: [PATCH] If a power saver block already exists, do nothing Before this change, when opening the player, both 'onPlayerOpen' and 'onPlayerPlay' would fire, which enabled, disabled, and re-enabled the power save blocker in quick succession. Instead, we just want to activate it once. --- src/main/power-save-blocker.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/main/power-save-blocker.js b/src/main/power-save-blocker.js index 97acd41e..26c85a67 100644 --- a/src/main/power-save-blocker.js +++ b/src/main/power-save-blocker.js @@ -13,7 +13,10 @@ var blockId = 0 * display. */ function enable () { - disable() // Stop the previous power saver block, if one exists. + if (electron.powerSaveBlocker.isStarted(blockId)) { + // If a power saver block already exists, do nothing. + return + } blockId = electron.powerSaveBlocker.start('prevent-display-sleep') log(`powerSaveBlocker.enable: ${blockId}`) } @@ -23,6 +26,7 @@ function enable () { */ function disable () { if (!electron.powerSaveBlocker.isStarted(blockId)) { + // If a power saver block does not exist, do nothing. return } electron.powerSaveBlocker.stop(blockId)