28317 - Pasting past text field's visible bounds doesn't trigger scrolling

Status: VERIFIED FIXED

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

Description

[Eli Goldberg]

* TITLE/SUMMARY
Pasting past text field's visible bounds doesn't trigger scrolling

* STEPS TO REPRODUCE
0) Launch Apprunner
1) View a web page with a text field, such as a Bugzilla bug (like this one)
2) Select a block of text from any application, say, 40-80 characters long.
3) Keep pasting the text into the "Additional Comments" text field.

* RESULT
 - What happened

The text field doesn't scroll downwards after the visible space is exceeded. This 
is also the case for a one-line text field, in which it doesn't scroll 
rightwards.

However, if you type in lots of random text, it will scroll.

 - What was expected 

Scrolling on paste that's equivalent to scrolling on text entry.

* REGRESSION

 - Occurs On
        Win32 Apprunner (2.17.00 AM optimized build [NT 4, Service Pack 5])
        Mac OS Apprunner (2.17.00 AM optimized build)		

 - Doesn't Occur On
        Mac OS Communicator 4.7 RTM
        Linux Apprunner (2.17.00 AM optimized build)
        Akkana Peck (if present in cubicle when reproducing on any platform)



* CONFIGURATIONS TESTED

- [Mac] Beige Power Mac G3 (266 MHz PowerPC 750), 96 MB RAM (VM on; 1 MB of VM 
used), 1024x768 (Thousands of Colors), Mac OS 8.6

- [Win32] Vectra VL (233 MHz P2), 96 MB RAM, 800x600 (True Color), NT 4.0 SP5.

- [Linux] Vectra VL (266 MHz P2), 96 MB RAM. Red Hat Linux 6.0 (GNOME).

Comment 1

[Eli Goldberg]

QA Assigning to self.

Comment 2

[Kathleen :Brade]

cc Kin; reassign to akkana

Comment 3

[kinmoz]

Perhaps we should move the nsTextEditorKeyListener::ScrollSelectionIntoView()
into nsEditor so that other methods like Paste() can use it.

Comment 4

[Akkana Peck]

Okay, I'll try that approach.

Comment 5

[Akkana Peck]

Fixed.  I didn't move the method from the key listener because it was just a
one-line call to a pres shell method (plus some mechanism for getting the pres
shell, etc.)

Comment 6

[Eli Goldberg]

Verified fixed in 3.28.00 builds on Mac OS/Win32, and 3.29.00 build on Linux.
Thanks!