Remove tabid from packageLogs

Author: Jyyjy

src/packageLogs.js

@@ -33,7 +33,6 @@ export let filterHandler = null;
 export let mapHandler = null;
 export let cbHandlers = {};
 
-
 /**
  * Assigns a handler to filter logs out of the queue.
  * @deprecated Use addCallbacks and removeCallbacks instead
@@ -105,29 +104,13 @@ export function initPackager(newLogs, newConfig) {
   intervalLog = null;
 }
 
-/**
- * Get the tabID from local storage
- */
-export function getTabId() {
-  const api = (typeof browser !== 'undefined') ? browser : chrome;
-  return new Promise((resolve, reject) => {
-    api.storage.local.get("tabId", function(result) {
-      if (result.tabId !== undefined) {
-        resolve(result.tabId);
-      } else {
-        reject('tabId not found');
-      }
-    });
-  });
-}
-
 /**
  * Transforms the provided HTML event into a log and appends it to the log queue.
  * @param  {Object} e         The event to be logged.
  * @param  {Function} detailFcn The function to extract additional log parameters from the event.
  * @return {boolean}           Whether the event was logged.
  */
-export async function packageLog(e, detailFcn) {
+export function packageLog(e, detailFcn) {
   if (!config.on) {
     return false;
   }
@@ -141,8 +124,6 @@ export async function packageLog(e, detailFcn) {
     (e.timeStamp && e.timeStamp > 0) ? config.time(e.timeStamp) : Date.now()
   );
 
-  const tabId = await getTabId();
-
   let log = {
     'target' : getSelector(e.target),
     'path' : buildPath(e),
@@ -162,7 +143,6 @@ export async function packageLog(e, detailFcn) {
     'toolVersion' : config.version,
     'toolName' : config.toolName,
     'useraleVersion': config.useraleVersion,
-    'tabId': tabId,
     'sessionID': config.sessionID,
   };
 
@@ -194,7 +174,7 @@ export async function packageLog(e, detailFcn) {
  * @param  {boolean} userAction     Indicates user behavior (true) or system behavior (false)
  * @return {boolean}           Whether the event was logged.
  */
-export async function packageCustomLog(customLog, detailFcn, userAction) {
+export function packageCustomLog(customLog, detailFcn, userAction) {
     if (!config.on) {
         return false;
     }
@@ -204,8 +184,6 @@ export async function packageCustomLog(customLog, detailFcn, userAction) {
         details = detailFcn();
     }
 
-    const tabId = await getTabId();
-
     const metaData = {
         'pageUrl': window.location.href,
         'pageTitle': document.title,
@@ -220,7 +198,6 @@ export async function packageCustomLog(customLog, detailFcn, userAction) {
         'toolVersion' : config.version,
         'toolName' : config.toolName,
         'useraleVersion': config.useraleVersion,
-        'tabId': tabId,
         'sessionID': config.sessionID
     };
 
@@ -266,7 +243,7 @@ export function extractTimeFields(timeStamp) {
  * @param {Object} e
  * @return boolean
  */
-export async function packageIntervalLog(e) {
+export function packageIntervalLog(e) {
     const target = getSelector(e.target);
     const path = buildPath(e);
     const type = e.type;
@@ -281,8 +258,6 @@ export async function packageIntervalLog(e) {
         intervalCounter = 0;
     }
 
-    const tabId = await getTabId();
-
     if (intervalID !== target || intervalType !== type) {
         // When to create log? On transition end
         // @todo Possible for intervalLog to not be pushed in the event the interval never ends...
@@ -307,7 +282,6 @@ export async function packageIntervalLog(e) {
             'toolVersion': config.version,
             'toolName': config.toolName,
             'useraleVersion': config.useraleVersion,
-            'tabId': tabId,
             'sessionID': config.sessionID
         };