Closed Bug 996605 Opened 10 years ago Closed 8 years ago

JavaScript Warning: "ReferenceError: reference to undefined property this.html[(intermediate value)]" {file: "chrome://browser/content/JSDOMParser.js" line: 642}

Categories

(Toolkit :: Reader Mode, defect)

Product:
Toolkit
Component:
Reader Mode
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: kats, Unassigned)

Details

Warning seen in logcat output on recent m-c code.
Mentor: margaret.leibovic
Component: Readability → Reader Mode
Product: Firefox for Android → Toolkit
Whiteboard: [good first bug][lang=js]
Actually, this probably isn't a good mentor bug.
Mentor: margaret.leibovic
Whiteboard: [good first bug][lang=js]
This is a strict warning that shows up for something like:

"foo"[4]

which returns undefined (no errors). Looking at the uses of this.html in the JSDOMParser code, I think we appropriately check for 'falsy' values, and adding length checks before the lookup will add complexity, avoid the warning but not change behaviour, and potentially slow down the (hot) code, so I don't think this is worth fixing.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.