From 2c88d1716e63be66946724c81667c8197ba80ac9 Mon Sep 17 00:00:00 2001
From: Dominik Broj <broj.dominik@gmail.com>
Date: Fri, 26 Oct 2018 20:05:01 +0200
Subject: [PATCH] refactor: return a promise instead of using a callback
 function

the callback-based api will soon be removed (https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/onMessage)
---
 keepassxc-browser/keepassxc-browser.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/keepassxc-browser/keepassxc-browser.js b/keepassxc-browser/keepassxc-browser.js
index ac6a313..e7bf94f 100755
--- a/keepassxc-browser/keepassxc-browser.js
+++ b/keepassxc-browser/keepassxc-browser.js
@@ -15,7 +15,7 @@ var _detectedFields = 0;
 // Element id's containing input fields detected by MutationObserver
 var _observerIds = [];
 
-browser.runtime.onMessage.addListener(function(req, sender, callback) {
+browser.runtime.onMessage.addListener(function(req, sender) {
     if ('action' in req) {
         if (req.action === 'fill_user_pass_with_specific_login') {
             if (cip.credentials[req.id]) {
@@ -63,10 +63,10 @@ browser.runtime.onMessage.addListener(function(req, sender, callback) {
             cipDefine.init();
         } else if (req.action === 'clear_credentials') {
             cipEvents.clearCredentials();
-            callback();
+            return Promise.resolve();
         } else if (req.action === 'activated_tab') {
             cipEvents.triggerActivatedTab();
-            callback();
+            return Promise.resolve();
         } else if (req.action === 'redetect_fields') {
             browser.runtime.sendMessage({
                 action: 'load_settings',