Lots of "TypeError: this._browsers is null: _getBrowserRefs@resource://app/modules/NewTabWebChannel.jsm:149" logspam during browser_remotenewtab_pageloads.js
VERIFIED
FIXED
in Firefox 52
Status
()
People
(Reporter: RyanVM, Assigned: mak)
Tracking
Firefox Tracking Flags
(firefox52 fixed, firefox-esr52 fixed, firefox53 fixed, firefox54 fixed)
Details
Attachments
(1 attachment)
59 bytes,
text/x-review-board-request
|
Felipe
:
review+
jcristau
:
approval-mozilla-aurora+
jcristau
:
approval-mozilla-beta+
|
Details |
This is a really spammy message while this test is running. Affects all current branches. https://treeherder.mozilla.org/logviewer.html#?job_id=74137204&repo=mozilla-beta&lineNumber=3516 Looking at the blame of the file, I'm not even sure who's around to look at this anymore. Kate, can you maybe take a look or suggest someone else who might?
Flags: needinfo?(khudson)
(Reporter) | ||
Updated•2 years ago
|
Summary: Lots of "ypeError: this._browsers is null: _getBrowserRefs@resource://app/modules/NewTabWebChannel.jsm:149" logspam during browser_remotenewtab_pageloads.js → Lots of "TypeError: this._browsers is null: _getBrowserRefs@resource://app/modules/NewTabWebChannel.jsm:149" logspam during browser_remotenewtab_pageloads.js
![]() |
||
Comment 1•2 years ago
|
||
Regression from bug 1320481. This is problematic because the error can occur so often that other failures aren't part of the error summary for a job on treeherder anymore and one has to open and search the raw log.
Blocks: 1320481
Flags: needinfo?(khudson) → needinfo?(mak77)
(Assignee) | ||
Updated•2 years ago
|
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Flags: needinfo?(mak77)
Comment hidden (mozreview-request) |
(Assignee) | ||
Comment 3•2 years ago
|
||
The problem is that frecency changes can send notifications quite late on shutdown, new tab receives the notification and tries to handle it. This is mostly a stopgap fix, I don't think we should go much further honestly.
Comment 4•2 years ago
|
||
mozreview-review |
Comment on attachment 8834048 [details] Bug 1336291 - Lots of 'TypeError: this._browsers is null' browser_remotenewtab_pageloads.js. https://reviewboard.mozilla.org/r/110146/#review111192
Attachment #8834048 -
Flags: review?(felipc) → review+
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/ec06d5263eb3 Lots of 'TypeError: this._browsers is null' browser_remotenewtab_pageloads.js. r=Felipe
Comment 6•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ec06d5263eb3
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox54: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
(Reporter) | ||
Comment 7•2 years ago
|
||
Confirmed that the spam is gone. Please request Aurora/Beta approval on this when you get a chance.
Status: RESOLVED → VERIFIED
Flags: needinfo?(mak77)
(Assignee) | ||
Comment 8•2 years ago
|
||
Comment on attachment 8834048 [details] Bug 1336291 - Lots of 'TypeError: this._browsers is null' browser_remotenewtab_pageloads.js. Approval Request Comment [Feature/Bug causing the regression]: bug 1320481 [User impact if declined]: No users impact, but lots of spam in test runs that impacts sheriffs and developers [Is this code covered by automated tests?]: not this specific fix [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: no [List of other uplifts needed for the feature/fix]: none [Is the change risky?]: no [Why is the change risky/not risky?]: simple early return [String changes made/needed]: none
Flags: needinfo?(mak77)
Attachment #8834048 -
Flags: approval-mozilla-beta?
Attachment #8834048 -
Flags: approval-mozilla-aurora?
Comment 9•2 years ago
|
||
Comment on attachment 8834048 [details] Bug 1336291 - Lots of 'TypeError: this._browsers is null' browser_remotenewtab_pageloads.js. early return to fix test log spam, aurora53+, beta52+
Attachment #8834048 -
Flags: approval-mozilla-beta?
Attachment #8834048 -
Flags: approval-mozilla-beta+
Attachment #8834048 -
Flags: approval-mozilla-aurora?
Attachment #8834048 -
Flags: approval-mozilla-aurora+
(Reporter) | ||
Comment 10•2 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/056b0e09741c
status-firefox53: affected → fixed
Flags: in-testsuite+
(Reporter) | ||
Comment 11•2 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/e6c3c117e08b
status-firefox52: affected → fixed
(Reporter) | ||
Comment 12•2 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-esr52/rev/e6c3c117e08b
status-firefox-esr52: --- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•