Closed
Bug 1530985
Opened 6 years ago
Closed 6 years ago
JavaScript Error: "listener[name] is not a function" from viewContextChanged@resource:///modules/UrlbarController.jsm
Categories
(Firefox :: Address Bar, defect, P2)
Firefox
Address Bar
Tracking
()
RESOLVED
FIXED
Firefox 67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: standard8, Assigned: mak)
References
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
I just spotted this in a test log from try:
08:46:47 INFO - Console message: [JavaScript Error: "listener[name] is not a function" {file: "resource:///modules/UrlbarController.jsm" line: 461}]
08:46:47 INFO - _notify@resource:///modules/UrlbarController.jsm:461:23
08:46:47 INFO - viewContextChanged@resource:///modules/UrlbarController.jsm:192:10
08:46:47 INFO - _on_TabSelect@resource:///modules/UrlbarInput.jsm:1112:21
08:46:47 INFO - handleEvent@resource:///modules/UrlbarInput.jsm:199:23
08:46:47 INFO - updateCurrentBrowser@chrome://browser/content/tabbrowser.js:1047:14
08:46:47 INFO - _setupEventListeners/<@chrome://browser/content/tabbrowser.js:4582:14
08:46:47 INFO - set selectedIndex@chrome://global/content/elements/tabbox.js:202:12
08:46:47 INFO - set selectedPanel@chrome://global/content/elements/tabbox.js:216:5
08:46:47 INFO - set_selectedIndex@chrome://global/content/bindings/tabbox.xml:176:15
08:46:47 INFO - set_selectedItem@chrome://global/content/bindings/tabbox.xml:201:39
08:46:47 INFO - set selectedTab@chrome://global/content/elements/tabbox.js:81:9
08:46:47 INFO - set selectedTab@chrome://browser/content/tabbrowser.js:266:5
08:46:47 INFO - _blurTab@chrome://browser/content/tabbrowser.js:3257:29
08:46:47 INFO - _beginRemoveTab@chrome://browser/content/tabbrowser.js:2957:10
08:46:47 INFO - removeTab@chrome://browser/content/tabbrowser.js:2865:15
08:46:47 INFO - Tester_waitForWindowsState@chrome://mochikit/content/browser-test.js:578:16
08:46:47 INFO - nextTest@chrome://mochikit/content/browser-test.js:902:10
08:46:47 INFO - async*testScope/test_finish/<@chrome://mochikit/content/browser-test.js:1407:25
08:46:47 INFO - run@chrome://mochikit/content/browser-test.js:1344:9
I suspect this is because bug 1496685 added a notification for onViewContextChanged
to the UrlbarController
but didn't add it as part of the listener function for UrlbarEventBufferer
.
We should either add it there or make it optional.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•6 years ago
|
||
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/60359928dd2c
Make Quantum Bar controller notifications optional. r=Standard8
Comment 3•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in
before you can comment on or make changes to this bug.
Description
•