Closed Bug 1699367 Opened 4 years ago Closed 4 years ago

Intermittent browser/components/extensions/test/browser/browser_ext_popup_select_in_oopif.js | Select popup has the correct x origin - Got 935, expected 1205

Categories

(WebExtensions :: Untriaged, defect, P5)

defect

Tracking

(firefox-esr78 unaffected, firefox86 unaffected, firefox87 unaffected, firefox88 disabled, firefox89 fixed)

RESOLVED FIXED
89 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- unaffected
firefox87 --- unaffected
firefox88 --- disabled
firefox89 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: hiro)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(1 file)

Filed by: nbeleuzu [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=333594657&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/PFK5ythhSTaxK0n_AwX2tg/runs/0/artifacts/public/logs/live_backing.log


[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_popup_select_in_oopif.js | Expect widget not to be overflowed - true == true - 
[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - Buffered messages finished
[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_popup_select_in_oopif.js | Select popup has the correct x origin - Got 935, expected 1205
[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - Stack trace:
[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - chrome://mochikit/content/browser-test.js:test_is:1359
[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - chrome://mochitests/content/browser/browser/components/extensions/test/browser/browser_ext_popup_select_in_oopif.js:testPopupSelectPopup:98
[task 2021-03-18T06:27:22.156Z] 06:27:22     INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_popup_select_in_oopif.js | Select popup has the correct y origin - ```
Has Regression Range: --- → yes

Okay, there are two kinds of failures, one is failing without Fission, the other one is failing with Fission.

Without Fission case, it's same as what I described about failures on TSAN in bug 1696718 comment 17, the popup position is simply wrong. I'd assume it's a pre-existing issue because the fix for bug 1696718 affects only for Fission and there was no automated test to test the popup position so that this new test just has started catching the pre-existing issue.

With Fission case, I think this is also somewhat related to the failures on TSAN what I described in bug 1696718 comment 17, with Fission SpecialPowers.spawn failed with an exception that the iframe's browsing context is null at that moment, which means, if the iframe's browsing context is null, we fail to propagate the extension's window position change into descendant OOP's BrowerParent because we descend down the browsing context tree when we propagate it.

Anyway, I am going to skip the test without Fission for now since the test is mainly for Fission and it looks like the Fission case failure doesn't often happen.

Assignee: nobody → hikezoe.birchill
Status: NEW → ASSIGNED
Flags: needinfo?(hikezoe.birchill)

The test basically was written for Fission, but unfortunately it fails
intermittently without Fission becasue, I assume, there was a pre-existing race
condition causing the failure without Fission, so we are going to skip the
test without Fission for now.

Pushed by hikezoe.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/da6d6bf1c78d Skip browser_ext_popup_select_in_oopif.js without Fission. r=kmag
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
Regressions: 1700285
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: