LoginManagerContent.jsm: TypeError: doc.documentElement is null

NEW
Unassigned

Status

()

Toolkit
Password Manager
2 years ago
24 days ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 1 bug, {regression, testcase})

Trunk
regression, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 wontfix, firefox-esr52 wontfix, firefox56 wontfix, firefox57 wontfix, firefox58 fix-optional)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Created attachment 8644513 [details]
testcase

JavaScript error: resource://gre/modules/LoginManagerContent.jsm, line 1221: TypeError: doc.documentElement is null

Probably a regression from https://hg.mozilla.org/mozilla-central/rev/2a9bb5102e19, which included the following change:

-      elements: [for (el of doc.querySelectorAll("input")) if (!el.form) el],
+      elements: [for (el of doc.documentElement.querySelectorAll("input")) if (!el.form) el],

(why?)
Flags: needinfo?(MattN+bmo)
This testcase still reproduces an error, though it's now:
> TypeError: rootElement is null  FormLikeFactory.jsm:78:1
>    for (let el of rootElement.querySelectorAll("input, select")) {
Has Regression Range: --- → yes
status-firefox42: affected → wontfix
status-firefox56: --- → wontfix
status-firefox57: --- → wontfix
status-firefox58: --- → fix-optional
status-firefox-esr52: --- → wontfix
Flags: needinfo?(MattN+bmo)
You need to log in before you can comment on or make changes to this bug.