Open Bug 940282 Opened 7 years ago Updated 6 years ago

View Source doesn't flag invalid character as error

Categories

(Core :: DOM: HTML Parser, defect)

defect
Not set
normal

Tracking

()

UNCONFIRMED

People

(Reporter: mike, Unassigned)

References

()

Details

Flags: needinfo?(hsivonen)
The basic problem here is that in Gecko, the code for reading a character is mere array access without checking the character for bogosity. I supposed we could look into using template magic to compile a different "read character" code path for View Source. But in that case, we'd then need a way to attach errors to individual characters.

So this is intentional in the sense that I knowingly didn't take the time to implement all this, but in principle, it would be nice to have code for this.
Flags: needinfo?(hsivonen)
(In reply to Henri Sivonen (:hsivonen) from comment #1)
> So this is intentional in the sense that I knowingly didn't take the time to
> implement all this, but in principle, it would be nice to have code for this.

OK, understood. I raised it in spirit with the same nice-to-have principle you mentioned, so I don't feel strongly about it (certainly not strongly enough to take time to write up a patch for it myself

So, given all that, I think moving this to resolved=wontfix would be fine. I don't think it's worth keeping open if there's not a need/activity to refine the behavior any time soon to add it.
You need to log in before you can comment on or make changes to this bug.