From cbbbedc0cc72ac13b89a022e27173f94c032e0b4 Mon Sep 17 00:00:00 2001 From: David Baker Date: Thu, 27 Oct 2016 12:29:10 +0100 Subject: [PATCH] Put try block in the right place --- src/vector/platform/ElectronPlatform.js | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/vector/platform/ElectronPlatform.js b/src/vector/platform/ElectronPlatform.js index 0c3852ef1f..af52a6399d 100644 --- a/src/vector/platform/ElectronPlatform.js +++ b/src/vector/platform/ElectronPlatform.js @@ -37,18 +37,18 @@ if (window && window.process && window.process && window.process.type === 'rende } export default class ElectronPlatform extends BasePlatform { - // this sometimes throws because electron is made of fail: - // https://github.com/electron/electron/issues/7351 - // For now, let's catch the error, but I suspect it may - // continue to fail and we might just have to accept that - // electron's remote RPC is a non-starter for now and use IPC - try { - setNotificationCount(count: number) { - super.setNotificationCount(count); + setNotificationCount(count: number) { + super.setNotificationCount(count); + // this sometimes throws because electron is made of fail: + // https://github.com/electron/electron/issues/7351 + // For now, let's catch the error, but I suspect it may + // continue to fail and we might just have to accept that + // electron's remote RPC is a non-starter for now and use IPC + try { remote.app.setBadgeCount(count); + } catch (e) { + console.error("Failed to set notification count", e); } - } catch (e) { - console.error("Failed to set notification count", e); } displayNotification(title: string, msg: string, avatarUrl: string): Notification {