From 28d3ba505d6a2722dfb55c8fc697ec81a4f423d6 Mon Sep 17 00:00:00 2001
From: Rimas Kudelis <rimas@datalogistiikka.fi>
Date: Fri, 12 Jul 2013 15:22:08 -0400
Subject: [PATCH] Don't set the dn variable unless it's needed.
---
plugins/newmail_notifier/newmail_notifier.js | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/plugins/newmail_notifier/newmail_notifier.js b/plugins/newmail_notifier/newmail_notifier.js
index 320382a..b2d7cf4 100644
--- a/plugins/newmail_notifier/newmail_notifier.js
+++ b/plugins/newmail_notifier/newmail_notifier.js
@@ -118,8 +118,7 @@
function newmail_notifier_test_desktop()
{
- var dn = window.webkitNotifications,
- txt = rcmail.gettext('testbody', 'newmail_notifier');
+ var txt = rcmail.gettext('testbody', 'newmail_notifier');
// W3C draft implementation (with fix for Chrome/Chromium)
try {
@@ -129,6 +128,7 @@
}
// webkit implementation
catch (e) {
+ var dn = window.webkitNotifications;
if (dn) {
if (!dn.checkPermission())
newmail_notifier_desktop(txt);
@@ -161,4 +161,4 @@
if (perm == 'granted')
newmail_notifier_test_desktop(); // Test again, which should show test message
});
-}
\ No newline at end of file
+}
--
Gitblit v1.9.1