135191 - gfx scrollframe sets curpos before maxpos when going back

Status: NEW

(Core :: XUL, defect)

Description

[Mike Pinkerton (not reading bugmail)]

when you go back/forward, we try to restore the previous scrollbar position. The
way the code does this however is a little skanky. It sets the curpos attribute
repeatedly during the page load but doesn't update the maxpos attribute until
it's totally done.

The problem with this is that this means that if it's communicating with a
native scrollbar, we're setting the curpos to greater than the maxpos for a
time. If things don't get cleaned up, this could render the native scrollbar
unusable depending on the impl of the native scrollbar.

Not a major issue, but makes it so that native scrollbar impls can't
sanity-check that curpos isn't greater than maxpos which could be dangerous.

Comment 1

[Mike Pinkerton (not reading bugmail)]

blizzard might be interested in this if they use the new xbl->native scrollbar
stuff i just did.

Comment 2

[Simon Paquet [:sipaq]]

happens on all operating systems

Comment 3

[Shu-yu Guo [:shu]]

Attachment: Hacky PeekReservedWord

This actually results in a small regression

Comment 4

[Shu-yu Guo [:shu]]

Comment on attachment 8858193 [details] [diff] [review]
Hacky PeekReservedWord

Oops, wrong bugno.