1465730 - arrowscrollbox scroll buttons are intermittently rendered as disabled or enabled when they should have the opposite state

Status: RESOLVED DUPLICATE of bug 1375012

(Toolkit :: UI Widgets, defect, P3)

Description

[:Paolo Amadini]

+++ This bug was initially created as a clone of Bug #1375012 +++

See bug 1368208 comment 56:

> I think I'll land this with 508816-1.xul disabled. The scroll buttons seem
> to suffer from some weird invalidation bug that I can reproduce even in
> nightly (e.g. in the all tabs list or in the bookmarks menu). My patch
> doesn't seem to make this worse but changes the timing such the reftest now
> exposes this bug.

I added some logging to _updateScrollButtonsDisabledState to see if it enables/disables the buttons as expected. It does do so, i.e. the buttons' disabled attributes get updated, but it seems that somehow this doesn't necessarily get rendered.

Comment 1

[:Paolo Amadini]

Note that once bug 1465457 is fixed, we may be able to use promiseDocumentFlushed and other privileged methods and see if they helps here.

Comment 2

[:Paolo Amadini]

Attachment: Bug 1465730 - Wait for a layout flush in the reftest for "arrowscrollbox" scrolling.

Review commit: https://reviewboard.mozilla.org/r/248184/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/248184/

Comment 3

[:Paolo Amadini]

Moved this together with the patch in bug 1375012 since these both depend on bug 1465457.