1851039 - Investigate if it is possible to reuse the text node when setting element.textContent

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement)

Description

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

Reusing the text node should be possible if it is owned only by the parent and possible primary frame and there are no mutation event listeners nor mutation observers.

Comment 1

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

Attachment: Bug 1851039 - Investigate if it is possible to reuse the text node when setting element.textContent, r=emilio

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e6370dc41077
Investigate if it is possible to reuse the text node when setting element.textContent, r=emilio

Comment 3

[Cristian Tuns]

Backed out for causing wpt failures in dynamic-mtext-like-001.html

• Backout link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-FAIL | /mathml/presentation-markup/tokens/dynamic-mtext-like-001.html | Testing http://web-platform.test:8000/mathml/presentation-markup/tokens/dynamic-mtext-like-001.html == http://web-platform.test:8000/mathml/presentation-markup/tokens/dynamic-mtext-like-001-ref.html

And ba failures: https://treeherder.mozilla.org/logviewer?job_id=427745896&repo=autoland

Comment 4

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

Interesting.

Comment 5

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

Smells like a layout bug, but will investigate

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3451691c483d
Investigate if it is possible to reuse the text node when setting element.textContent, r=emilio

Comment 7

[Sandor Molnar[:smolnar]]

Backed out for causing mochitest failures in accessible/tests/browser/e10s/browser_caching_description.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/9544b885e5a8df34cd7105ec16907c5db432d095

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | accessible/tests/browser/e10s/browser_caching_description.js | Test timed out

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7fd1c7032f52
Investigate if it is possible to reuse the text node when setting element.textContent, r=emilio

Comment 9

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/7fd1c7032f52