this.widget is null 1 ext-browserAction.js:523 in Browser console when click on New Tab Button

RESOLVED FIXED

Status

()

defect
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: alice0775, Unassigned)

Tracking

57 Branch
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox55 unaffected, firefox56 unaffected, firefox57 fixed)

Details

(Whiteboard: [reserve-photon-structure])

(Reporter)

Description

2 years ago
STR:
click on New Tab Button
(Reporter)

Updated

2 years ago
Blocks: 1396412
(Reporter)

Updated

2 years ago
Component: New Tab Page → Tabbed Browser
This happens for both opening and closing a tab:

Stack trace for opening a tab:

this.widget is null  ext-browserAction.js:523
    updateWindow chrome://browser/content/ext-browserAction.js:523:9
    build/< chrome://browser/content/ext-browserAction.js:231:40
    emit resource://gre/modules/ExtensionUtils.jsm:258:40
    handleEvent chrome://browser/content/ext-browser.js:133:7
    updateCurrentBrowser chrome://browser/content/tabbrowser.xml:1308:15
    onselect chrome://browser/content/browser.xul:1:44
    set_selectedIndex chrome://browser/content/tabbrowser.xml:8343:13
    set_selectedPanel chrome://global/content/bindings/tabbox.xml:694:13
    set_selectedIndex chrome://global/content/bindings/tabbox.xml:412:15
    set_selectedItem chrome://global/content/bindings/tabbox.xml:444:34
    set_selectedTab chrome://global/content/bindings/tabbox.xml:114:15
    set_selectedTab chrome://browser/content/tabbrowser.xml:3761:11
    loadOneTab chrome://browser/content/tabbrowser.xml:1739:15
    openLinkIn chrome://browser/content/utilityOverlay.js:458:26
    openUILinkIn chrome://browser/content/utilityOverlay.js:207:3
    BrowserOpenTab chrome://browser/content/browser.js:2312:3
    oncommand chrome://browser/content/browser.xul:1:1


Stack trace for closing a tab:

this.widget is null  ext-browserAction.js:523
    updateWindow chrome://browser/content/ext-browserAction.js:523:9
    build/< chrome://browser/content/ext-browserAction.js:231:40
    emit resource://gre/modules/ExtensionUtils.jsm:258:40
    handleEvent chrome://browser/content/ext-browser.js:133:7
    updateCurrentBrowser chrome://browser/content/tabbrowser.xml:1308:15
    onselect chrome://browser/content/browser.xul:1:44
    set_selectedIndex chrome://browser/content/tabbrowser.xml:8343:13
    set_selectedPanel chrome://global/content/bindings/tabbox.xml:694:13
    set_selectedIndex chrome://global/content/bindings/tabbox.xml:412:15
    set_selectedItem chrome://global/content/bindings/tabbox.xml:444:34
    set_selectedTab chrome://global/content/bindings/tabbox.xml:114:15
    set_selectedTab chrome://browser/content/tabbrowser.xml:3761:11
    _blurTab chrome://browser/content/tabbrowser.xml:3281:15
    _beginRemoveTab chrome://browser/content/tabbrowser.xml:3031:13
    removeTab chrome://browser/content/tabbrowser.xml:2938:18
    removeCurrentTab chrome://browser/content/tabbrowser.xml:2897:13
    BrowserCloseTabOrWindow chrome://browser/content/browser.js:2411:3
    oncommand chrome://browser/content/browser.xul:1:1

Comment 2

2 years ago
I have been getting this error on clicking a tab (FF56)

ReferenceError: ignoreWindow is not defined
Stack trace:
@resource://gre/modules/commonjs/sdk/ui/state.js:223:1
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
forward/<@resource://gre/modules/commonjs/sdk/event/utils.js:85:31
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
receive@resource://gre/modules/commonjs/sdk/event/utils.js:115:5
next@resource://gre/modules/commonjs/sdk/event/utils.js:37:24
filter/<@resource://gre/modules/commonjs/sdk/event/utils.js:54:7
transform/<@resource://gre/modules/commonjs/sdk/event/utils.js:44:29
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
receive@resource://gre/modules/commonjs/sdk/event/utils.js:115:5
next@resource://gre/modules/commonjs/sdk/event/utils.js:37:24
map/<@resource://gre/modules/commonjs/sdk/event/utils.js:61:60
transform/<@resource://gre/modules/commonjs/sdk/event/utils.js:44:29
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
forward/<@resource://gre/modules/commonjs/sdk/event/utils.js:85:31
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
forward/<@resource://gre/modules/commonjs/sdk/event/utils.js:85:31
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
forward/<@resource://gre/modules/commonjs/sdk/event/utils.js:85:31
emitOnObject@resource://gre/modules/commonjs/sdk/event/core.js:110:7
emit@resource://gre/modules/commonjs/sdk/event/core.js:86:38
listener@resource://gre/modules/commonjs/sdk/event/dom.js:41:29
updateCurrentBrowser@chrome://browser/content/tabbrowser.xml:1262:15
onselect@chrome://browser/content/browser.xul:1:44
set_selectedIndex@chrome://global/content/bindings/tabbox.xml:672:13
set_selectedPanel@chrome://global/content/bindings/tabbox.xml:691:13
set_selectedIndex@chrome://global/content/bindings/tabbox.xml:409:15
set_selectedItem@chrome://global/content/bindings/tabbox.xml:441:34
set_selectedTab@chrome://global/content/bindings/tabbox.xml:111:15
set_selectedTab@chrome://browser/content/tabbrowser.xml:3638:11
ssi_restoreWindow@resource:///modules/sessionstore/SessionStore.jsm:3344:11
initializeWindow@resource:///modules/sessionstore/SessionStore.jsm:1173:7
onBeforeBrowserWindowShown@resource:///modules/sessionstore/SessionStore.jsm:1267:7
ssi_observe@resource:///modules/sessionstore/SessionStore.jsm:764:9
onLoad@chrome://browser/content/browser.js:1358:5
onload@chrome://browser/content/browser.xul:1:1

core.js:100
The issue in comment 2 is probably unrelated.
Whiteboard: [photon-structure] → [photon-structure] [triage]

Comment 4

2 years ago
This issue seems to be related to the custom way that the screenshot system addon (https://github.com/mozilla-services/screenshots/blob/master/addon/) is using internally to move the browserAction defined in the manifest.json file of the embedded webextension:

- https://github.com/mozilla-services/screenshots/blob/master/addon/webextension/manifest.json.template#L14-L21) 

into the new Photon Library toolbar menu:

- https://github.com/mozilla-services/screenshots/blob/8011e7bd7f7e47decdc10dabb33a58580ec0456e/addon/bootstrap.js#L269-L278

By removing the CustomizableUI widget with id "screenshots_mozilla_org-browser-action" right after its creation from ext-browserAction.js:

- http://searchfox.org/mozilla-central/rev/999385a5e8c2d360cc37286882508075fc2078bd/browser/components/extensions/ext-browserAction.js#133,233

`this.widget` will always be null for the screenshot hybrid system add-on, and so the listener subscribed to "tab-select":

- http://searchfox.org/mozilla-central/rev/999385a5e8c2d360cc37286882508075fc2078bd/browser/components/extensions/ext-browserAction.js#230-231

raises the exception from Comment 1 every time a tab is selected (and as a side-effect also when a tab is created or removed).
Flags: qe-verify?
Priority: -- → P3
Whiteboard: [photon-structure] [triage] → [reserve-photon-structure]
I get this error whenever switching tabs. It is adding spam to the console that makes development more difficult.

Comment 6

2 years ago
(In reply to Dave Townsend [:mossop] from comment #5)
> I get this error whenever switching tabs. It is adding spam to the console
> that makes development more difficult.

This is fixed in the screenshots repo, here: https://github.com/mozilla-services/screenshots/commit/234fffcf1c7a524ce54256b70009570731d59c57 .

It looks like this will be merged to m-c as part of bug 1399615.
Blocks: 1399615
No longer blocks: 1396412
Flags: qe-verify? → qe-verify-

Comment 7

2 years ago
bug 1399615 landed, so this is fixed.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Iteration: --- → 57.3 - Sep 19
Priority: P3 → P1
You need to log in before you can comment on or make changes to this bug.