New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 688379 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Last visit 29 days ago
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Node.normalize() leaves empty textnode

Reported by alexande...@gmail.com, Feb 3 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0

Steps to reproduce the problem:
1. Create a Node (e.g. a 'div') with more than 1 textnode of which one (the last one?) is empty
2. call node.normalize()

What is the expected behavior?
There should only be 1 textnode in the surrounding Node.

What went wrong?
There is an empty textnode in the surrounding node.

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 55.0.2883.87  Channel: stable
OS Version: Ubuntu
Flash Version: Shockwave Flash 24.0 r0

Checked that it works correctly in Firefox.

 
chromeNormalizeBug.html
329 bytes View Download

Comment 1 by ajha@chromium.org, Feb 6 2017

Labels: Needs-Milestone

Comment 2 by ajha@chromium.org, Feb 6 2017

Components: Blink

Comment 3 by kochi@chromium.org, Feb 8 2017

Components: -Blink Blink>DOM
Routing to DOM team.
Cc: hdodda@chromium.org
Labels: Needs-Feedback
Tested on ubuntu 14.04 using chrome stable M56 #56.0.2924.87 and observed as attached in screencast.

Steps followed :
1. Opened the attached html file and called "node.normalize" in web console.

Attached screencast for reference.

@alexander.popiak--Could you please check the attached screencast and let us know if we have missed out anything in reproducing the issue .Also , please try in latest chrome stable and if you can still reproduce the issue , help us by providing the screencast.

Thanks!
688379.ogv
3.8 MB View Download
The script included in the attached HTML already executes "node.normalize()" (so it's not strictly necessary to do it again on the console) and the issue can then be seen in the inspector as the node having more than one child text node (of which one is empty).
Project Member

Comment 6 by sheriffbot@chromium.org, Feb 16 2017

Labels: -Needs-Feedback Needs-Review
Owner: hdodda@chromium.org
Thank you for providing more feedback. Adding requester "hdodda@chromium.org" for another review and adding "Needs-Review" label for tracking.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by tkent@chromium.org, Feb 17 2017

Cc: tkent@chromium.org
Labels: -Needs-Review -Needs-Milestone Needs-Feedback
I couldn't reproduce the issue.
After loading the attached chromeNormalizeBug.html, node.childNodes.length is 1 with Google Chrome 56.0.2924.87.

alexander.popiak@, can you check with the latest Google Chrome?

Comment 8 by tkent@chromium.org, Feb 17 2017

NextAction: 2017-03-03
Owner: dominicc@chromium.org
Status: WontFix (was: Unconfirmed)
I can't reproduce this with 58.0.3013.3 (Official Build) dev (64-bit) either.
Update: I cannot reproduce the issue in a way that makes node.childNodes.length greater than 1, but the dev tools show empty text nodes in the div. Maybe it's a dev tools bug?
See attached images for dev tools session that shows the empty nodes (even though node.childNodes.length is 0)
chromium_dev_tools_bug.png
105 KB View Download
Components: -Blink>DOM Platform>DevTools
Labels: -Needs-Feedback -Hotlist-Interop
NextAction: ----
Owner: ----
Status: Untriaged (was: WontFix)
Yeah, it sounds like a DevTools bug.

I confirmed:
 - Loading chromeNormalizeBug.html, then opening DevTools didn't show an empty Text node.
 - Loading chromeNormalizeBug.html, opening DevTools, reloading chromeNormalizeBug.html, showed an empty Text node.

Components: -Platform>DevTools Platform>DevTools>Authoring
Owner: lushnikov@chromium.org
Status: Assigned (was: Untriaged)
Status: WontFix (was: Assigned)
Closing as WontFix due to lack priority / resources.

Sign in to add a comment