Closed Bug 1194396 Opened 9 years ago Closed 9 years ago

Remove unnecessary RemoveAllRanges call in nsDocumentViewer::SelectAll()

Categories

(Core :: DOM: Selection, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: nika, Assigned: nika)

References

Details

Attachments

(1 file)

      No description provided.
Currently nsDocumentViewer::SelectAll calls RemoveAllRanges, immediately followed by SelectAllChildren. SelectAllChildren itself calls Collapse, which handles removing the currently selected ranges from the document. Thus, this call to RemoveAllRanges is unnecessary, and can be removed.
Assignee: nobody → michael
Attachment #8647658 - Flags: review?(ehsan)
Comment on attachment 8647658 [details] [diff] [review]
Remove unnecessary RemoveAllRanges call in nsDocumentViewer::SelectAll()

We don't need this anymore for bug 571294
Attachment #8647658 - Flags: review?(ehsan)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: