Closed Bug 13476 Opened 26 years ago Closed 26 years ago

GetDocumentLength should not do a SelectAll

Categories

(Core :: DOM: Editor, defect, P3)

All
Mac System 8.5
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: sfraser_bugs, Assigned: buster)

Details

GetDocumentLength() uses a rather bizarre way of getting the length of the document in characters, by doing a SelectAll, then iterating over nodes. Apart from the potential performance aspects, this has the unfortunate side-effect of scrolling the document. Joe says that it would be better simply to iterate over the children of the <body>.
Status: NEW → ASSIGNED
should take me 10 minutes to rewrite this.
Whiteboard: fix in my tree, waiting for a green tree
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Whiteboard: fix in my tree, waiting for a green tree
Target Milestone: M11
fixed
can I get steps to verify this one? IF its code level issue, can you verify it? thanks!
it's code. I verified the code. To verify that I didn't break anything, play with a text control that has a maxlength attribute. try all combinations of adding and removing content. type, delete, backspace, select-and-type, cut, copy, paste, etc.
Status: RESOLVED → REOPENED
I think something broke...see 13923... it could be unrelated...I'm re-opening this one just as a place holder... go ahead and kick it back to me if its a separate issue..
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
13923 is unrelated
Status: RESOLVED → VERIFIED
Looks fixed...tried various mentioned operations.... verified in 9/15 build.
You need to log in before you can comment on or make changes to this bug.