106565 - SetInnerHTML always uses quirks mode

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect, P1)

Description

[Boris Zbarsky [:bzbarsky]]

Setting innerHTML on an element always parses the string in quirks mode.

Testcase coming up.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: testcase

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: better testcase -- nothing should show up in the content area

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: Patch to fix this

Comment 4

[Boris Zbarsky [:bzbarsky]]

Harish, this involves some parser changes that you should take a look at...
Could you, please?

Comment 5

[harishd]

Boris: Sure I will take a look....but not right away :-( .... since I'm
currently focusing on a performance issue.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Ok.  This is not a big deal, really.  Few sites depend on strict comment parsing
in innerHTML.  :)

Comment 7

[harishd]

Comment on attachment 55009 [details] [diff] [review]
Patch to fix this

may the default mode should be auto. With that r=harishd.

Comment 8

[Boris Zbarsky [:bzbarsky]]

Attachment: patch that uses GetOwnerDocument instead of nodeinfo in nsRange (no parser changes from last patch)

Comment 9

[Johnny Stenback (:jst)]

Comment on attachment 58059 [details] [diff] [review]
patch that uses GetOwnerDocument instead of nodeinfo in nsRange (no parser changes from last patch)

sr=jst

Comment 10

[Boris Zbarsky [:bzbarsky]]

checked in

Comment 11

[Hixie (not reading bugmail)]

v using test cases