1602620 - ubuntu1804 - toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor before_start top - got 78, expected -4

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: egao [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=280332609&repo=try
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/I-LQh1gERma0_Xnsi0-Kzw/runs/0/artifacts/public/logs/live_backing.log

---

suite: mochitest-chrome-1proc
chunk: 2

context:
[task 2019-12-09T18:45:40.593Z] 18:45:40 INFO - TEST-START | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml
[task 2019-12-09T18:45:40.662Z] 18:45:40 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-12-09T18:45:40.664Z] 18:45:40 INFO - Buffered messages logged at 18:45:40
[task 2019-12-09T18:45:40.664Z] 18:45:40 INFO - must wait for load
[task 2019-12-09T18:45:40.664Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | initial left
[task 2019-12-09T18:45:40.665Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | initial top
[task 2019-12-09T18:45:40.665Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor top + 8 left
[task 2019-12-09T18:45:40.666Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor top + 8 top
[task 2019-12-09T18:45:40.666Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor left + 6, top - 10 left
[task 2019-12-09T18:45:40.667Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor left + 6, top - 10 top
[task 2019-12-09T18:45:40.667Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor before_start left
[task 2019-12-09T18:45:40.668Z] 18:45:40 INFO - Buffered messages finished
[task 2019-12-09T18:45:40.672Z] 18:45:40 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor before_start top - got 78, expected -4
[task 2019-12-09T18:45:40.673Z] 18:45:40 INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:325:16
[task 2019-12-09T18:45:40.674Z] 18:45:40 INFO - checkCoords@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml:68:5
[task 2019-12-09T18:45:40.674Z] 18:45:40 INFO - popupshown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml:44:14
[task 2019-12-09T18:45:40.675Z] 18:45:40 INFO - onpopupshown@chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml:1:1
[task 2019-12-09T18:45:40.676Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move button2 left
[task 2019-12-09T18:45:40.679Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move button2 top
[task 2019-12-09T18:45:40.680Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor end_before left
[task 2019-12-09T18:45:40.681Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move anchor end_before top
[task 2019-12-09T18:45:40.682Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move button2 left + 5, top + 4 left
[task 2019-12-09T18:45:40.686Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move button2 left + 5, top + 4 top
[task 2019-12-09T18:45:40.689Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move to button1 screen with offset left
[task 2019-12-09T18:45:40.690Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move to button1 screen with offset top
[task 2019-12-09T18:45:40.691Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move button2 after screen left
[task 2019-12-09T18:45:40.692Z] 18:45:40 INFO - TEST-PASS | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | move button2 after screen top
[task 2019-12-09T18:45:40.693Z] 18:45:40 INFO - GECKO(7500) | MEMORY STAT | vsize 3281MB | residentFast 379MB | heapAllocated 151MB
[task 2019-12-09T18:45:40.695Z] 18:45:40 INFO - TEST-OK | toolkit/content/tests/chrome/test_popup_moveToAnchor.xhtml | took 71ms

Comment 1

[Edwin Takahashi (:egao | infrequent contributor)]

:bgrins - this is another permafail observable on ubuntu1804 test image. It would be appreciated if you can redirect the ni for this as well.

I do wonder if this is also related to changes in the desktop environment between ubuntu1804 and previous distributions.

To push to try using the ubuntu1804 image, use ./mach try fuzzy --ubuntu-bionic and select the test-linux jobs as normal.

Comment 2

[Brian Grinstead [:bgrins]]

(In reply to Edwin Takahashi (:egao, :etakahashi) from comment #1)

:bgrins - this is another permafail observable on ubuntu1804 test image. It would be appreciated if you can redirect the ni for this as well.

I do wonder if this is also related to changes in the desktop environment between ubuntu1804 and previous distributions.

To push to try using the ubuntu1804 image, use ./mach try fuzzy --ubuntu-bionic and select the test-linux jobs as normal.

Also redirecting this one.

Comment 3

[Neil Deakin]

While I can't reproduce this problem locally, the error suggests that the popup is a bit taller than the test handles. The -4 expected value suggests that it is 4 pixels too tall. It so, then it should be fixable by just increasing the margin on the two buttons by 10 pixels or so just to be safe.

Comment 4

[Neil Deakin]

Attachment: Bug 1602620, fix margins and add some debugging to see why these tests fail on ubuntu18.04

Comment 5

[Neil Deakin]

The patch here adds some debugging to see what the issue is with bug as well as 1602616 and 1602624.

The try command above doesn't work for me, so I can look at the results if someone could try it out.

Comment 6

[Edwin Takahashi (:egao | infrequent contributor)]

I will apply the patch and report back with the try push.

EDIT: try push: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=259b5f8690253e8b437f2e60a5712083293f81da

Comment 7

[Intermittent Failures Robot]

1 failures in 4867 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 1

Platform breakdown:

• linux64: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1602620&startday=2019-12-09&endday=2019-12-15&tree=all

Comment 8

[Neil Deakin]

The try run above doesn't appear to have the patch applied.

Comment 9

[Edwin Takahashi (:egao | infrequent contributor)]

(In reply to Neil Deakin from comment #8)

The try run above doesn't appear to have the patch applied.

Interesting, I will try again.

EDIT: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&tier=1%2C2%2C3&revision=190e69a89f6a8bf75acf187f1c0782a27ba7c23e

Comment 10

[Neil Deakin]

Attachment: Bug 1602620, adjust the margin to accomodate the different theme in ubuntu 18.04, this was causing the popup to not fit aligned above the button and get flipped to the other side of the button, r=bgrins

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d2e0e35baeff
adjust the margin to accomodate the different theme in ubuntu 18.04, this was causing the popup to not fit aligned above the button and get flipped to the other side of the button, r=bgrins

Comment 12

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/d2e0e35baeff