Skip to content

innerText setter should replace existing text node - #3493

Merged
Ms2ger merged 1 commit into
masterfrom
innertext-single-text-node
Jan 17, 2017
Merged

innerText setter should replace existing text node#3493
Ms2ger merged 1 commit into
masterfrom
innertext-single-text-node

Conversation

@zcorpan

@zcorpan zcorpan commented Aug 18, 2016

Copy link
Copy Markdown
Member

This was referenced Aug 18, 2016
Open
Merged
@ayg ayg closed this Aug 28, 2016
@ayg
ayg deleted the innertext-single-text-node branch August 28, 2016 13:58
@zcorpan
zcorpan restored the innertext-single-text-node branch August 28, 2016 20:10
@zcorpan zcorpan reopened this Aug 28, 2016
@ayg ayg closed this Oct 26, 2016
@ayg
ayg deleted the innertext-single-text-node branch October 26, 2016 17:38
@ayg
ayg restored the innertext-single-text-node branch October 26, 2016 18:04
@ayg ayg reopened this Oct 26, 2016
@wpt-stability-bot

wpt-stability-bot commented Oct 26, 2016

Copy link
Copy Markdown

Firefox

Testing revision 2f961df
Starting 10 test iterations
All results were stable

All results

/innerText/setter.html

Subtest Results
OK
Newlines convert to
in 
 element
 PASS
Simplest possible test PASS
> preserved PASS
Tabs preserved PASS
< preserved PASS
' preserved PASS
Existing
deleted		 PASS
Newlines convert to
in white-space:pre element		 PASS
Null characters preserved PASS
innerText not supported on MathML elements PASS
Newline/CR pair converts to
in non-white-space:pre element		 PASS
innerText not supported on SVG elements PASS
& preserved PASS
CRs convert to
in 
 element
 PASS
Assigning the empty string PASS
Whitespace not compressed PASS
Existing text deleted PASS
CRs convert to
in white-space:pre element		 PASS
Trailing whitespace preserved PASS
Assigning undefined PASS
CR/CR pair converts to two
s in non-white-space:pre element		 PASS
Newline/newline pair converts to two
s in non-white-space:pre element		 PASS
Newlines convert to
in non-white-space:pre elements		 PASS
CRs convert to
in non-white-space:pre elements		 PASS
Leading whitespace preserved PASS
Assigning null PASS
" preserved PASS

@wpt-stability-bot

wpt-stability-bot commented Oct 26, 2016

Copy link
Copy Markdown

Chrome

Testing revision 2f961df
Starting 10 test iterations
All results were stable

All results

/innerText/setter.html

Subtest Results
OK
Newlines convert to
in 
 element
 PASS
Simplest possible test PASS
> preserved PASS
Tabs preserved PASS
< preserved PASS
' preserved PASS
Existing
deleted		 PASS
Newlines convert to
in white-space:pre element		 PASS
Null characters preserved PASS
innerText not supported on MathML elements PASS
Newline/CR pair converts to
in non-white-space:pre element		 PASS
innerText not supported on SVG elements PASS
& preserved PASS
CRs convert to
in 
 element
 PASS
Assigning the empty string PASS
Whitespace not compressed PASS
Existing text deleted FAIL
CRs convert to
in white-space:pre element		 PASS
Trailing whitespace preserved PASS
Assigning undefined PASS
CR/CR pair converts to two
s in non-white-space:pre element		 PASS
Newline/newline pair converts to two
s in non-white-space:pre element		 PASS
Newlines convert to
in non-white-space:pre elements		 PASS
CRs convert to
in non-white-space:pre elements		 PASS
Leading whitespace preserved PASS
Assigning null PASS
" preserved PASS

@zcorpan
zcorpan requested a review from Ms2ger January 17, 2017 10:27
@Ms2ger
Ms2ger merged commit aae3e1b into master Jan 17, 2017
@Ms2ger
Ms2ger deleted the innertext-single-text-node branch January 17, 2017 10:36
@Ms2ger
Ms2ger removed their request for review January 17, 2017 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

innerText status:needs-review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@zcorpan @wpt-stability-bot @Ms2ger @sideshowbarker @ayg @wpt-pr-bot