Skip to content
This repository was archived by the owner on Feb 19, 2025. It is now read-only.

Commit 8a54754

Browse files
committed
add tabId to content script logs
1 parent 5b94d26 commit 8a54754

File tree

5 files changed

+24
-19
lines changed

5 files changed

+24
-19
lines changed

build/UserALEWebExtension/background.js

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -415,7 +415,7 @@ function createVersionParts(count) {
415415
* limitations under the License.
416416
*/
417417

418-
var browser$1 = detect();
418+
var browserInfo = detect();
419419
var logs$1;
420420
var config$1;
421421

@@ -706,8 +706,8 @@ function selectorizePath(path) {
706706
}
707707
function detectBrowser() {
708708
return {
709-
'browser': browser$1 ? browser$1.name : '',
710-
'version': browser$1 ? browser$1.version : ''
709+
'browser': browserInfo ? browserInfo.name : '',
710+
'version': browserInfo ? browserInfo.version : ''
711711
};
712712
}
713713

@@ -1142,7 +1142,7 @@ function dispatchTabMessage(message) {
11421142
});
11431143
});
11441144
}
1145-
browser.runtime.onMessage.addListener(function (message) {
1145+
browser.runtime.onMessage.addListener(function (message, sender, sendResponse) {
11461146
switch (message.type) {
11471147
case CONFIG_CHANGE:
11481148
options(message.payload);
@@ -1151,7 +1151,11 @@ browser.runtime.onMessage.addListener(function (message) {
11511151

11521152
// Handles logs rerouted from content and option scripts
11531153
case ADD_LOG:
1154-
log(message.payload);
1154+
var log$1 = message.payload;
1155+
if ("tab" in sender && "id" in sender.tab) {
1156+
log$1["tabId"] = sender.tab.id;
1157+
}
1158+
log(log$1);
11551159
break;
11561160
default:
11571161
console.log('got unknown message type ', message);

build/UserALEWebExtension/content.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,7 @@ function createVersionParts(count) {
405405
* limitations under the License.
406406
*/
407407

408-
var browser$1 = detect();
408+
var browserInfo = detect();
409409
var logs$1;
410410
var config$1;
411411

@@ -720,8 +720,8 @@ function selectorizePath(path) {
720720
}
721721
function detectBrowser() {
722722
return {
723-
'browser': browser$1 ? browser$1.name : '',
724-
'version': browser$1 ? browser$1.version : ''
723+
'browser': browserInfo ? browserInfo.name : '',
724+
'version': browserInfo ? browserInfo.version : ''
725725
};
726726
}
727727

@@ -1100,7 +1100,6 @@ function options(newConfig) {
11001100
// browser is defined in firefox, but chrome uses the 'chrome' global.
11011101
var browser = browser || chrome;
11021102
function rerouteLog(log) {
1103-
console.log(log);
11041103
browser.runtime.sendMessage({
11051104
type: ADD_LOG,
11061105
payload: log

build/UserALEWebExtension/options.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,7 @@ function createVersionParts(count) {
405405
* limitations under the License.
406406
*/
407407

408-
var browser$1 = detect();
408+
var browserInfo = detect();
409409
var logs$1;
410410
var config$1;
411411

@@ -720,8 +720,8 @@ function selectorizePath(path) {
720720
}
721721
function detectBrowser() {
722722
return {
723-
'browser': browser$1 ? browser$1.name : '',
724-
'version': browser$1 ? browser$1.version : ''
723+
'browser': browserInfo ? browserInfo.name : '',
724+
'version': browserInfo ? browserInfo.version : ''
725725
};
726726
}
727727

@@ -1100,7 +1100,6 @@ function options(newConfig) {
11001100
// browser is defined in firefox, but chrome uses the 'chrome' global.
11011101
var browser = browser || chrome;
11021102
function rerouteLog(log) {
1103-
console.log(log);
11041103
browser.runtime.sendMessage({
11051104
type: ADD_LOG,
11061105
payload: log

build/userale-2.4.0.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -418,7 +418,7 @@
418418
* limitations under the License.
419419
*/
420420

421-
var browser = detect();
421+
var browserInfo = detect();
422422
var logs$1;
423423
var config$1;
424424

@@ -745,8 +745,8 @@
745745
}
746746
function detectBrowser() {
747747
return {
748-
'browser': browser ? browser.name : '',
749-
'version': browser ? browser.version : ''
748+
'browser': browserInfo ? browserInfo.name : '',
749+
'version': browserInfo ? browserInfo.version : ''
750750
};
751751
}
752752

src/UserALEWebExtension/background.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ function dispatchTabMessage(message) {
4343
});
4444
}
4545

46-
browser.runtime.onMessage.addListener(function (message) {
46+
browser.runtime.onMessage.addListener(function (message, sender, sendResponse) {
4747
switch (message.type) {
4848
case MessageTypes.CONFIG_CHANGE:
4949
userale.options(message.payload)
@@ -52,7 +52,11 @@ browser.runtime.onMessage.addListener(function (message) {
5252

5353
// Handles logs rerouted from content and option scripts
5454
case MessageTypes.ADD_LOG:
55-
userale.log(message.payload);
55+
let log = message.payload;
56+
if("tab" in sender && "id" in sender.tab) {
57+
log["tabId"] = sender.tab.id;
58+
}
59+
userale.log(log);
5660
break;
5761

5862
default:
@@ -75,7 +79,6 @@ function packageDetailedTabLog(tab, data, type) {
7579
// Attach Handlers for tab events
7680
// https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/tabs
7781
browser.tabs.onActivated.addListener((activeInfo) => {
78-
browser.storage.local.set({ "tabId": activeInfo.tabId });
7982
packageTabLog(activeInfo.tabId, activeInfo, "tabs.onActivated");
8083
});
8184

0 commit comments

Comments
 (0)