Closed
Bug 1411490
Opened 8 years ago
Closed 7 years ago
[Win7] Intermittent editor/reftests/xul/number-3.xul == editor/reftests/xul/number-ref.xul | image comparison, max difference: 255, number of differing pixels: 56
Categories
(Core :: Layout, defect, P5)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell fixed:other])
Attachments
(2 files, 1 obsolete file)
19.28 KB,
image/png
|
Details | |
2.35 KB,
patch
|
gbrown
:
review-
|
Details | Diff | Splinter Review |
Filed by: philringnalda [at] gmail.com
https://treeherder.mozilla.org/logviewer.html#?job_id=139365134&repo=autoland
https://queue.taskcluster.net/v1/task/NWXfadFHRTSwZE6dx3Banw/runs/0/artifacts/public/logs/live_backing.log
https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/NWXfadFHRTSwZE6dx3Banw/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 3•8 years ago
|
||
There are 32 failures over the last 7 days.
Most of them occur on Windows 7 (pgo, debug, opt) and windows7-32-stylo-disabled (opt)
Here is a recent log example: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=144044657&lineNumber=2742
:jet can you please take a look?
Thank you!
Flags: needinfo?(bugs)
Whiteboard: [stockwell needswork]
Comment 4•8 years ago
|
||
here is the reftest analyzer:
https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/DoXHEW1OS2Snh_Qqz5Nj6w/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1
looking at it in detail, I see there is an input field with a 0 in it- the 0 is shifted in position slightly
Comment hidden (Intermittent Failures Robot) |
Comment 6•8 years ago
|
||
(In reply to Stefan Hindli [:stefan_hindli] from comment #3)
> There are 32 failures over the last 7 days.
> Most of them occur on Windows 7 (pgo, debug, opt) and
> windows7-32-stylo-disabled (opt)
>
> Here is a recent log example:
> https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-
> central&job_id=144044657&lineNumber=2742
>
> :jet can you please take a look?
>
> Thank you!
This test should be a 100% fail on Windows:
https://searchfox.org/mozilla-central/source/editor/reftests/xul/reftest.list#21
Masayuki: should these turn into fails-if(winWidget)?
fails-if(windowsDefaultTheme&&/^Windows\x20NT\x20(5\.[12]|6\.[012]|10\.0)/.test(http.oscpu))
Flags: needinfo?(bugs) → needinfo?(masayuki)
Comment hidden (Intermittent Failures Robot) |
Comment 8•8 years ago
|
||
Sorry for the delay to reply.
The "0" is positioned differently. Could be sub-pixel positioning issue on Windows?
Flags: needinfo?(masayuki) → needinfo?(jfkthame)
Comment 9•8 years ago
|
||
Oops, the previous image was tiff.
Attachment #8929943 -
Attachment is obsolete: true
Comment 10•8 years ago
|
||
The tests here are annotated with fails-if(windowsDefaultTheme...), and have been ever since they first landed in bug but it appears from checking the reftest log that the windowsDefaultTheme flag is not always being set as expected.
When I look at the log from a passing run, I see that it has \"windowsDefaultTheme\":true, whereas the failing log has \"windowsDefaultTheme\":false. Do we really expect some of our reftest jobs to occasionally run under different theme conditions than others? Or is the problem that the windowsDefaultTheme flag is a bit unreliable?
I don't really know anything about how this works in the reftest harness, but bug 1050229 suggests Markus may be somewhat familiar with it.... passing needinfo? along to him.
Flags: needinfo?(jfkthame) → needinfo?(mstange)
Comment hidden (Intermittent Failures Robot) |
Comment 12•8 years ago
|
||
In the last 7 days there are 31 failures. They occur on Windows 7.
A recent log example: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=147429434&lineNumber=2727
:jet, can you please take a look?
Flags: needinfo?(bugs)
Whiteboard: [stockwell unknown] → [stockwell needswork]
Comment 13•8 years ago
|
||
(In reply to Narcis Beleuzu [:NarcisB] from comment #12)
> In the last 7 days there are 31 failures. They occur on Windows 7.
> A recent log example:
> https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-
> inbound&job_id=147429434&lineNumber=2727
>
> :jet, can you please take a look?
Let's disable these for Win32 and move on.
Flags: needinfo?(mstange)
Flags: needinfo?(bugs)
Comment 14•8 years ago
|
||
Attachment #8932875 -
Flags: review?(gbrown)
![]() |
||
Comment 15•8 years ago
|
||
Comment on attachment 8932875 [details] [diff] [review]
disable the tests on windows
Review of attachment 8932875 [details] [diff] [review]:
-----------------------------------------------------------------
I don't think this will be effective: I checked several recent failure logs, and all had windowsDefaultTheme=false.
I think we need to follow-up on comment 10 to better understand windowsDefaultTheme, or disable on all win32.
Attachment #8932875 -
Flags: review?(gbrown) → review-
Comment 16•8 years ago
|
||
ok, this is a great observations Geoff and I see that all failures are windowsDefaultTheme:False and passing instances are True. We set this value here:
https://searchfox.org/mozilla-central/source/layout/tools/reftest/manifest.jsm#576
sandbox.windowsDefaultTheme = g.containingWindow.matchMedia("(-moz-windows-default-theme)").matches;
while we don't run stylo-disabled, historically there has been only one instance of this failure on stylo-disabled which did have the theme- either way there is either a bug in:
1) the environment we are running in
2) how we query for moz-windows-default-theme
I see bug 1343049 and when these failures were greatly reduced a fix was landed (Nov 23rd). This indicates that it is related to #1 and that #1 isn't fixed yet.
Depends on: 1343049
Comment hidden (Intermittent Failures Robot) |
Comment 18•8 years ago
|
||
There have been 37 failures in the last week
This is occurring only on the Windows 7 platform.
As for the build type, I see 20 occurrences on opt, 16 on pgo and 1 on debug.
Here is a link from a recent log file: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=150953315&lineNumber=2761
And a link to the reftest-analyzer: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/ST3H4OToSZu4EsO53LtxUA/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1
Please also take a look at Comment 16
Flags: needinfo?(bugs)
Whiteboard: [stockwell unknown] → [stockwell needswork]
Comment hidden (Intermittent Failures Robot) |
Comment 20•8 years ago
|
||
(In reply to Tiberius Oros[:tiberius_oros] from comment #18)
> Please also take a look at Comment 16
Comment 16 says that this bug requires that bug 1343049 gets fixed.
Flags: needinfo?(bugs)
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 27•8 years ago
|
||
Summary:
There have been 37 failures in the last week.
All of the failures are on Windows 7 and on pgo and opt build types.
:jet, please take a look at this after Bug 1343049 is fixed. Thank you.
Whiteboard: [stockwell unknown] → [stockwell needswork]
Comment 28•8 years ago
|
||
(In reply to Tiberius Oros[:tiberius_oros] from comment #27)
> :jet, please take a look at this after Bug 1343049 is fixed. Thank you.
I'm fairly certain that you will come find me if this bug still happens after bug 1343049 is fixed :)
Flags: needinfo?(bugs)
Comment hidden (Intermittent Failures Robot) |
Comment 30•7 years ago
|
||
We have 34 failures in the last 7 days.
They occur mostly on Windows7 (opt and pgo) and the rest o them on windows10-64(pgo), windows7-32-nightly (opt).
Reftest analyzer: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/IF7-gHm8QZugEXd917b00w/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1
Recent log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=155909359&lineNumber=8369
Waiting for bug 1343049 to be fixed.
Comment hidden (Intermittent Failures Robot) |
Comment 32•7 years ago
|
||
In the last 7 days there are 34 failures.
They occur on Windows and the affected builds type are pgo, opt and debug.
Reftest analyzer: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/WZvPIwc7STScjKtFAoU0ZA/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1
Recent log: https://treeherder.mozilla.org/logviewer.html#?repo=autoland&job_id=158773406&lineNumber=2666
Still waiting for bug 1343049 to be fixed.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 36•7 years ago
|
||
This was a Win7 incorrect-theme issue; but the failures noted in comment 35 are all from Linux. So they are actually something quite different, and should have been filed as a separate bug.
Updated•7 years ago
|
Summary: Intermittent editor/reftests/xul/number-3.xul == editor/reftests/xul/number-ref.xul | image comparison, max difference: 255, number of differing pixels: 56 → [Win7] Intermittent editor/reftests/xul/number-3.xul == editor/reftests/xul/number-ref.xul | image comparison, max difference: 255, number of differing pixels: 56
Comment 37•7 years ago
|
||
this was either fixed by run-by-manifests or changes made to the image.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [stockwell unknown] → [stockwell fixed:other]
You need to log in
before you can comment on or make changes to this bug.
Description
•