Open
Bug 1375012
Opened 7 years ago
Updated 2 years ago
arrowscrollbox scroll buttons are intermittently rendered as disabled or enabled when they should have the opposite state
Categories
(Toolkit :: XUL Widgets, defect, P3)
Toolkit
XUL Widgets
Tracking
()
NEW
Tracking | Status | |
---|---|---|
firefox57 | --- | wontfix |
firefox58 | --- | wontfix |
firefox59 | --- | fix-optional |
People
(Reporter: dao, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
text/x-review-board-request
|
Details |
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.
Updated•7 years ago
|
status-firefox57:
--- → wontfix
Priority: -- → P3
Updated•7 years ago
|
status-firefox58:
--- → wontfix
status-firefox59:
--- → fix-optional
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
With the timing aligned, the test doesn't fail anymore in a tryserver run of 20 rebuilds on all platforms:
https://treeherder.mozilla.org/testview.html?repo=try&revision=1fe08c6d023035d382df88a2b4e08317c85ab53f
This can unblock bug 1457719, even though the reftest is still running unprivileged until bug 1465457 is fixed.
Blocks: 1457719
Reporter | ||
Comment 3•6 years ago
|
||
I'm still seeing what I wrote in comment 0, so I think we have a legitimate bug here, not just a test issue. If we can enable the test regardless, that's fine with me, but we'll still want to track the underlying issue I filed this bug for.
Comment 4•6 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #3)
> I'm still seeing what I wrote in comment 0, so I think we have a legitimate
> bug here, not just a test issue. If we can enable the test regardless,
> that's fine with me, but we'll still want to track the underlying issue I
> filed this bug for.
Sounds good, I cloned this to bug 1465730.
Summary: arrowscrollbox scroll buttons are intermittently rendered as disabled or enabled when they should have the opposite state → Align the timing of "508816-1.xul" and "508816-1-ref.xul"
Updated•6 years ago
|
Attachment #8982133 -
Flags: review?(dao+bmo)
Updated•6 years ago
|
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•