Remove isElementContentWhitespace

RESOLVED FIXED in mozilla10

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: annevk, Assigned: Ms2ger)

Tracking

({addon-compat, dev-doc-complete})

Trunk
mozilla10
addon-compat, dev-doc-complete
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Text.isElementContentWhitespace is no longer part of the DOM standard.
(Reporter)

Comment 1

6 years ago
WebKit does not support this attribute. I filed a bug on Opera for removing it there (along with replaceWholeText()).
(Assignee)

Comment 2

6 years ago
Created attachment 560927 [details] [diff] [review]
Patch v1
Assignee: nobody → Ms2ger
Status: NEW → ASSIGNED
Attachment #560927 - Flags: review?(jonas)
(Assignee)

Updated

6 years ago
Depends on: 688190

Updated

6 years ago
Keywords: dev-doc-needed
OS: Mac OS X → All
Hardware: x86 → All
(Assignee)

Comment 3

6 years ago
Jonas?
Comment on attachment 560927 [details] [diff] [review]
Patch v1

Hrm.. this actually seemed like a useful function. It always feels silly to me when all browsers have to have state or functionality internally that isn't exposed to the webpage.

So I'd be in favor of keeping this in spec/implementations. But if people prefer to remove it then I won't stand in the way.
Attachment #560927 - Flags: review?(jonas) → review+
(Reporter)

Comment 5

6 years ago
The current name does not make any sense. I would be open to renaming it. E.g. Text.isWhiteSpace. I used an uppercase S for consistency with white-space in CSS.
I'd be fine with that.
(Reporter)

Comment 7

6 years ago
I filed http://www.w3.org/Bugs/Public/show_bug.cgi?id=14443 on adding Text.isWhiteSpace to DOM4, but Simon rightly asks what the use case is and I have no idea.
(Assignee)

Comment 8

6 years ago
https://hg.mozilla.org/mozilla-central/rev/292803c8c406
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Version: unspecified → Trunk
(Assignee)

Updated

6 years ago
Keywords: addon-compat
I updated:
https://developer.mozilla.org/en/Firefox_10_for_developers
https://developer.mozilla.org/en/DOM/document.xmlEncoding
https://developer.mozilla.org/en/DOM/document
Keywords: dev-doc-needed → dev-doc-complete
Oops, wrong bug (sorry for the spam).
Keywords: dev-doc-complete → dev-doc-needed
This time, I updated:
https://developer.mozilla.org/en/Firefox_10_for_developer
https://developer.mozilla.org/En/DOM/Text.isElementContentWhitespace
https://developer.mozilla.org/En/DOM/Text
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.