12066 - (feature) Need to expose Selection repaint method.

Status: VERIFIED FIXED

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

Description

[kinmoz]

We need to add a public methods to nsIDOMSelection that allow us to scroll the
anchor or focus node into view.

Comment 1

[kinmoz]

Whoops, wrong description.

We need to add a public method to nsIDOMSelection that allows users to repaint
only the selection.

Comment 2

[kinmoz]

Accepting bug. Setting milestone to M10.

Comment 3

[kinmoz]

Fix for bugs #7153, #10673, #12066, #12067, and #12793 checked in:

    editor/base/nsEditorEventListeners.cpp    revision 1.89
    editor/base/nsEditorEventListeners.h      revision 1.115
        - Added code to scroll the selection into view
          after processing key events.
        - Commented out the hack that redraws the entire
          view when the focus is gained and lost. Replaced
          the hack code with calls to RepaintSelection().

    layout/base/public/nsIFrameSelection.h    revision 1.115
    layout/base/public/nsIPresShell.h         revision 3.46
    layout/html/base/src/nsPresShell.cpp      revision 3.167
        - Added ScrollSelectionIntoView() and RepaintSelection() methods.

    layout/base/src/nsRangeList.cpp    revision 1.123
        - Added implementation for ScrollSelectionIntoView() and
          RepaintSelection().
        - Check for NULL primary frame in GetFocusNodeRect().

Comment 4

[Eli Goldberg]

Developer tracking bug; rubber-stamping as fixed.