13407 - [DOGFOOD] gfx text fields scramble when scrolling

Status: VERIFIED FIXED

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

Description

[Chris McAfee]

linux, apprunner.
Go to bug 12902, scroll the cc list to the right,
then to the left.  Things redraw funny.

Comment 1

[kinmoz]

Accepting bug.

Comment 2

[kinmoz]

Changing milestone of all Selection Scrolling bugs to M12.

Comment 3

[kinmoz]

Reassigning to [e-mail].

Akkana, Pavlov, and I got together to look at this.

Turns out it's a problem in the gtk implementation of the method
nsScrollbar::SetPosition(). The view scrolling code expects SetPosition() to
move the scrollbar thumb without causing a scroll to happen, but the gtk
implementation does trigger a scroll, so what ends up happening is that
nsScrollingView::ScrollTo() ends up scrolling the view twice because it calls
both SetPosition() and Scroll().

To fix this, the code in SetPosition() needs to be modified so that it doesn't
emmit a scroll signal.

I'm not sure of the urgency to fix this bug since things seem to work fine when
using Gfx Scrollbars. I believe the plan right now is to have all platforms
switch over to using Gfx Scrollbars before the beta.

Comment 4

[kinmoz]

Marking this bug as dogfood.

Comment 5

[Chris McAfee]

Until gfx scrollbar primetime, we might be able
to check the pref and conditionally not do one
of the SetPostion() or Scroll() calls.

Comment 6

[leger]

Putting on the PDT+ radar.

Comment 7

[Sebastian Späth]

*** Bug 17256 has been marked as a duplicate of this bug. ***

Comment 8

[rubydoo123]

*** Bug 19169 has been marked as a duplicate of this bug. ***

Comment 9

[rubydoo123]

Attachment: a very simple form test case, an input element with default text

Comment 10

[rubydoo123]

*** Bug 18353 has been marked as a duplicate of this bug. ***

Comment 11

[Stuart Parmenter]

this is fixed with gfx scrollbars on.  i've added a comment to the gtk code with
a pointer to this bug incase we ever get around to fixing it for native
scrollbars

Comment 12

[sujay]

will verify in 12/1 build.

Comment 13

[sujay]

looks fixed to me....mcafee reopen if you're stil not seeing it fixed..

verified in 12/1 build.

Comment 14

[cpratt]

*** Bug 19516 has been marked as a duplicate of this bug. ***