Open Bug 1340786 Opened 8 years ago Updated 2 years ago

toolkit/content/tests/chrome/test_largemenu.html fails when run on windows7 vm (instead of hardware)

Categories

(Toolkit :: XUL Widgets, defect, P5)

defect

Tracking

()

Tracking Status
firefox56 --- wontfix
firefox57 --- wontfix
firefox58 --- wontfix
firefox59 --- ?

People

(Reporter: jmaher, Unassigned)

References

Details

Attachments

(1 file)

an small bit from the log [0]: 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | open with scrolling bottom 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | open with scrolling overflow 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu scroll position 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu scroll position 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | open small again top 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | open small again bottom 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | open small again overflow 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu scroll position 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement (1, 1) x 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement (1, 1) y 13:25:35 INFO - Buffered messages finished 13:25:35 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/chrome/test_largemenu.xul | menu movement (100, 8000) x - got 1280, expected 100 13:25:35 INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:271:5 13:25:35 INFO - is@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:164:24 13:25:35 INFO - testPopupMovement@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:343:3 13:25:35 INFO - popupShown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:98:12 13:25:35 INFO - onpopupshown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:1:1 13:25:35 INFO - Not taking screenshot here: see the one that was previously logged 13:25:35 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/chrome/test_largemenu.xul | menu movement (100, 8000) y - got 864, expected 688 13:25:35 INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:271:5 13:25:35 INFO - is@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:164:24 13:25:35 INFO - testPopupMovement@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:344:3 13:25:35 INFO - popupShown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:98:12 13:25:35 INFO - onpopupshown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:1:1 13:25:35 INFO - Not taking screenshot here: see the one that was previously logged 13:25:35 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/chrome/test_largemenu.xul | menu movement (6000, 100) x - got 2805, expected 1205 13:25:35 INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:271:5 13:25:35 INFO - is@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:164:24 13:25:35 INFO - testPopupMovement@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:357:3 13:25:35 INFO - popupShown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:98:12 13:25:35 INFO - onpopupshown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/window_largemenu.xul:1:1 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement (6000, 100) y 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement left is empty after moving 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement top is empty after moving 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement set left and top x 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement set left and top y 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement move after set left and top x 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement move after set left and top y 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement left is set after moving 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement top is set after moving 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement move after set left and top x to -1 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement move after set left and top y to -1 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement left is not set after moving to -1 13:25:35 INFO - TEST-PASS | toolkit/content/tests/chrome/test_largemenu.xul | menu movement top is not set after moving to -1 13:25: Fixing this bug will allow us to move over all of mochitest-chrome to a VM and reducing the load we have on hardware! https://archive.mozilla.org/pub/firefox/try-builds/davidp99@gmail.com-657209ff283794a666a2302a78a9368ca2ab0674/try-win32-debug/try_win7_vm_gfx-debug_test-mochitest-chrome-3-bm128-tests1-windows-build5.txt.gz
Neil, can you help us figure this out? This happens both on opt and debug in the same ways.
Flags: needinfo?(enndeakin)
The numbers suggest something odd about the screen configuration. There is only one screen configured, right? Do you have a screenshot of the failure?
Flags: needinfo?(enndeakin)
The screenshot suggests some kind of rendering issue; the popup is drawn multiple times yet there is only of them. The popup should be drawn near the bottom of the screen.
any tips on how to debug this? would some more logging in the test case help? Any system settings that we could do that could help (windows registry settings, etc.)? From looking at the logs it appears we have 1280x1024 resolution both on hardware and vm- maybe that is not what it really is, although finding screenshots for win7-hw failures is hard- I did find one and it looks to be similar sized screenshot: http://mozilla-releng-blobs.s3.amazonaws.com/blobs/mozilla-inbound/sha512/50f30a6dc342d6ea8ff90808ed45f8489e9863d784f1460daf0c110044f96220f6a5537da700b9e0319746e13ec005e8827386631379b88fe3b11502902b7ec8
There's a difference between the two screenshots. One of the versions of Windows is running without using all graphic fanciness (no transparencies). Can we assume that has nothing to do? (since my assumption is that we have failures in both machines with graphics and without). Neil: anything we could try to investigate a bit more?
Flags: needinfo?(enndeakin)
Does this fail the same everytime? The part of the test that is failing is trying to move a menu to a position off the bottom of the screen, but it should be pushed up to be visible along the bottom edge of the screen. You could check to see what the various coordinates and sizes of the screen are as those are used to compute the values in the test that is failing.
Flags: needinfo?(enndeakin)
Attached patch Extra debugging (deleted) — Splinter Review
This adds that extra debugging info.
Assignee: nobody → enndeakin
Assignee: enndeakin → nobody
Neil, this is the only test preventing us from moving mochitest-chrome to a VM; is it ok to skip this on windows? maybe we move this to the 'clipboard' suite which always runs on hardware?
Flags: needinfo?(enndeakin)
Did you try running with the debugging patch from comment 8? Otherwise, I would suggest trying to run this in the clipboard set if that works, as this test checks some platform-specific behaviour that would go untested otherwise.
Flags: needinfo?(enndeakin)
my try push is here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c8aecad1255a4164f9ecf8a169f3c4b381fe3c87&filter-tier=1&filter-tier=2&filter-tier=3&filter-searchStr=c3&exclusion_profile=false there is a failure on a gpu instance and a green on a non-gpu instance (target platform). I did some retriggers, lets see what shakes out!
ok, so looking at the c3 failures they are all on gpu instances (not our target), Neil, is there data you can harvest from this patch? Should I push without your patch and see if things are 'magically' fixed?
Flags: needinfo?(enndeakin)
The results show that the screen information is correct yet the popup is indeed appearing horizontally offscreen. It does sound like the is specific to the machine configuration. I can produce a much more detailed debugging patch if desired with lots of extra logging if we want to investigate further. Do I need to do anything special to push my own try-builds for testing?
Flags: needinfo?(enndeakin)
ok, here is a try push with no extra patches: https://treeherder.mozilla.org/#/jobs?repo=try&revision=65b74b3926cac3bb10e95908e7a2f1ced668fc15&filter-tier=1&filter-tier=2&filter-tier=3 try pushes are as simple as: try: -b o -p win32 -u mochitest-chrome -t none that will run win7 hardware, and win7 vm- but the win7-vm jobs are excluded, so you need to get that showing on treeherder with a toggle near the top of the webpage. you can see that all the regular instances are passing. Should we debug why the gpu instances fail? I think we can move forward and port these to windows7-vm; I would recommend closing this bug as worksforme, but happy to help debug more.
Debugging shows that the failing machines have two screens configured: The first is at position 0,0 with a size of 1280,1024 The second is to the right of the first at 1280,0 and ends at 1600,1200. That has a size of 320x1200 which is an unusual configuration. Passing machines have only the first.
Flags: needinfo?(jmaher)
oh, interesting find, Q: do you know why the gpu instances in aws have a second screen available to the win7 session? Possibly this could hurt us in some areas, but maybe help us in others.
Flags: needinfo?(jmaher) → needinfo?(q)
I don't know why it would be inconsistent with some machines having it and some not. However, there are two video devices a cirrus card and an nvidia card on a GPU deivce. There may be some tweaking to the fakemon script to make sure that the second screen is disabled. Q
Flags: needinfo?(q)
Greg, another one for you to look at when you get bored :)
Assignee: nobody → gmierz2
(In reply to Q from comment #17) > I don't know why it would be inconsistent with some machines having it and > some not. However, there are two video devices a cirrus card and an nvidia > card on a GPU deivce. There may be some tweaking to the fakemon script to > make sure that the second screen is disabled. > > Q Q, can you take a look to make sure that the fakemon script is doing the right thing here?
Flags: needinfo?(q)
Apologies folks got my wires crossed here and thought this was being taken care of in Rob's TC work. However, Looking back at old bugs I can see this is a buildbot request. Are we still seeing this failure ?
Flags: needinfo?(q)
Priority: -- → P5
Summary: toolkit/content/tests/chrome/test_largemenu.xul fails when run on windows7 vm (instead of hardware) → toolkit/content/tests/chrome/test_largemenu.html fails when run on windows7 vm (instead of hardware)
Assignee: gmierz2 → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: