Closed Bug 4057 Opened 26 years ago Closed 26 years ago

nsRangeList::Clear() not resetting focus and anchor nodes

Categories

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

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: mozeditor, Assigned: mozeditor)

Details

(Whiteboard: 3-28 waiting on reporter/owner to verify)

Clear() is just chucking out the dom ranges that represent the selection, but it is not also clearing out the frame level selection information. There are three possibilities here: 1) leave it like it is 2) have it clear out the presentation shells concept of focus frame and anchor frame 3) do 2) plus actually walk the frame list and unset any selection bits in the frames. I'm examining the usage and intent of Clear() to determine which of these is correct.
Status: NEW → ASSIGNED
Target Milestone: M4
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Summary: nsRangeList::Clear() not resetting focus and anchor frames → nsRangeList::Clear() not resetting focus and anchor nodes
ooops, misunderstood Steve at first. He meant that Clear() wasn't setting focus and anchor NODES, not frames. Fixed.
Whiteboard: waiting on reporter/owner to verify
If the reporter or owner could verify this bug and make it go away for all of us I would be much obliged. thank you
Whiteboard: waiting on reporter/owner to verify → 3-28 waiting on reporter/owner to verify
I've asked Steve Clark to verify this. Although I made the "fix", I was never able to see a problem in the first place, so I want Steve to verify.
you don't see any problem because I had worked around it. I looked at the code you checked in and it seems fine.
Status: RESOLVED → VERIFIED
i'm taking the above statements to mean this bug fix is as verified as its ever going to get. marking VERIFIED
Per a request from Selection and Search component eng (mjudge) and qa (elig), moving all "Selection and Search" bugs to new "Selection" component. Original "Selection and Search" component will be retired.
You need to log in before you can comment on or make changes to this bug.