1470017 - Consider to not change prototype when adopting nodes

Status: NEW

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

Description

[Olli Pettay [:smaug][[e-mail]]]

https://www.w3.org/Bugs/Public/show_bug.cgi?id=20567

Comment 1

[Boris Zbarsky [:bzbarsky]]

We should consider doing some measurements on the memory aspect here somehow...

Comment 2

[Bobby Holley (:bholley)]

(In reply to Boris Zbarsky [:bz] (no decent commit message means r-) from comment #1)
> We should consider doing some measurements on the memory aspect here
> somehow...

I already did, in [1].

So I'm not really in a hurry to do this. If we're going to do anything here, we might as well start with adopting Blink's model for document.open(), since the rationale in [2] is that we can drop transplanting from the spec if we do both of these things.

[1] https://www.w3.org/Bugs/Public/show_bug.cgi?id=20567#c56
[2] https://www.w3.org/Bugs/Public/show_bug.cgi?id=20567#c74

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The severity field for this bug is set to S3. However, the following bug duplicate has higher severity:

• Bug 1821790: S2

:hsinyi, could you consider increasing the severity of this bug to S2?

For more information, please visit auto_nag documentation.

Comment 6

[Hsin-Yi Tsai (she/her) [:hsinyi]]

According to the know webcompat report, I am not quite sure if I see the rationale of Bug 1821790 being S2. Kagami, would you kindly answer this needinfo from comment 5 for me, as you have better ideas about this issue? Thank you.

Comment 7

[Kagami Rosylight [:saschanaz] (they/them)]

I have no idea, S3 looks fine to me. I'll pass that question to mccr8 who set it.

Comment 8

[Andrew McCreight [:mccr8]]

I thought it was a regression, but this is apparently old behavior.