1855518 - Failures in dom/tests/mochitest/geolocation/test_native_provider.html in M-xorig

Status: RESOLVED FIXED

(GeckoView Graveyard :: Sandboxing, defect, P2)

Description

[[:owlish] 🦉 PST]

This test fails on all Android builds with both isolateEverything and isolateHighValue isolation strategies (fission.webContentIsolationStrategy set to 1 or 2 respectively).

Failures:

[task 2023-09-26T00:09:44.481Z] 00:09:44  WARNING -  TEST-UNEXPECTED-FAIL | dom/tests/mochitest/geolocation/test_native_provider.html | error callback should not have been called
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  SimpleTest.ok@https://example.org/tests/SimpleTest/SimpleTest.js:426:16
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  test_maximumAge/</<@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:28:9
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  PositionErrorCallback*test_maximumAge/<@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:24:27
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  test_maximumAge@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:23:9
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  async*nextTick/<@https://example.org/tests/SimpleTest/SimpleTest.js:2189:34
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  nextTick@https://example.org/tests/SimpleTest/SimpleTest.js:2233:11
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  setTimeout handler*SimpleTest_setTimeoutShim@https://example.org/tests/SimpleTest/SimpleTest.js:922:41
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  add_task@https://example.org/tests/SimpleTest/SimpleTest.js:2137:17
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  @https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:18:9
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  add_task | Leaving test_maximumAge
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  add_task | Entering test_highAccuracy
[task 2023-09-26T00:09:44.481Z] 00:09:44  WARNING -  TEST-UNEXPECTED-FAIL | dom/tests/mochitest/geolocation/test_native_provider.html | error callback should not have been called on low accuracy call
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  SimpleTest.ok@https://example.org/tests/SimpleTest/SimpleTest.js:426:16
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  test_highAccuracy/lowAccuracy</<@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:44:9
[task 2023-09-26T00:09:44.481Z] 00:09:44     INFO -  PositionErrorCallback*test_highAccuracy/lowAccuracy<@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:41:27
[task 2023-09-26T00:09:44.482Z] 00:09:44     INFO -  test_highAccuracy@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:40:29
[task 2023-09-26T00:09:44.482Z] 00:09:44     INFO -  async*nextTick/<@https://example.org/tests/SimpleTest/SimpleTest.js:2189:34
[task 2023-09-26T00:09:44.482Z] 00:09:44     INFO -  async*nextTick@https://example.org/tests/SimpleTest/SimpleTest.js:2233:11
[task 2023-09-26T00:09:44.482Z] 00:09:44     INFO -  setTimeout handler*SimpleTest_setTimeoutShim@https://example.org/tests/SimpleTest/SimpleTest.js:922:41
[task 2023-09-26T00:09:44.482Z] 00:09:44     INFO -  add_task@https://example.org/tests/SimpleTest/SimpleTest.js:2137:17
[task 2023-09-26T00:09:44.483Z] 00:09:44     INFO -  @https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:18:9
[task 2023-09-26T00:09:44.483Z] 00:09:44  WARNING -  TEST-UNEXPECTED-FAIL | dom/tests/mochitest/geolocation/test_native_provider.html | error callback should not have been called on high accuracy call
[task 2023-09-26T00:09:44.483Z] 00:09:44     INFO -  SimpleTest.ok@https://example.org/tests/SimpleTest/SimpleTest.js:426:16
[task 2023-09-26T00:09:44.483Z] 00:09:44     INFO -  test_highAccuracy/highAccuracy</<@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:54:9
[task 2023-09-26T00:09:44.484Z] 00:09:44     INFO -  PositionErrorCallback*test_highAccuracy/highAccuracy<@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:51:27
[task 2023-09-26T00:09:44.484Z] 00:09:44     INFO -  test_highAccuracy@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:50:30
[task 2023-09-26T00:09:44.484Z] 00:09:44     INFO -  async*nextTick/<@https://example.org/tests/SimpleTest/SimpleTest.js:2189:34
[task 2023-09-26T00:09:44.484Z] 00:09:44     INFO -  async*nextTick@https://example.org/tests/SimpleTest/SimpleTest.js:2233:11
[task 2023-09-26T00:09:44.484Z] 00:09:44     INFO -  setTimeout handler*SimpleTest_setTimeoutShim@https://example.org/tests/SimpleTest/SimpleTest.js:922:41
[task 2023-09-26T00:09:44.484Z] 00:09:44     INFO -  add_task@https://example.org/tests/SimpleTest/SimpleTest.js:2137:17
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  @https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:18:9
[task 2023-09-26T00:09:44.485Z] 00:09:44  WARNING -  TEST-UNEXPECTED-FAIL | dom/tests/mochitest/geolocation/test_native_provider.html | lower accuracy calls should not out perform high accuracy calls
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  SimpleTest.ok@https://example.org/tests/SimpleTest/SimpleTest.js:426:16
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  test_highAccuracy@https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:65:7
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  async*nextTick/<@https://example.org/tests/SimpleTest/SimpleTest.js:2189:34
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  async*nextTick@https://example.org/tests/SimpleTest/SimpleTest.js:2233:11
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  setTimeout handler*SimpleTest_setTimeoutShim@https://example.org/tests/SimpleTest/SimpleTest.js:922:41
[task 2023-09-26T00:09:44.485Z] 00:09:44     INFO -  add_task@https://example.org/tests/SimpleTest/SimpleTest.js:2137:17
[task 2023-09-26T00:09:44.486Z] 00:09:44     INFO -  @https://example.org/tests/dom/tests/mochitest/geolocation/test_native_provider.html?currentTestURL=dom%2Ftests%2Fmochitest%2Fgeolocation%2Ftest_native_provider.html&closeWhenDone=1&showTestReport=false&expected=pass:18:9

example try push
logcat from that try push

Comment 1

[[:owlish] 🦉 PST]

Hey Olivia, I think you are familiar with this area - is this something for our team to fix?

Comment 2

[Olivia Hall [:olivia]]

I think we probably will need to do the initial investigation to see if it is in DOM or in Android.

It could also be something going on with the test mock location provided by geo.provider.testing in Fission too.

In junit tests, we mock the location provider, while in mochitest we use this one: geo.provider.testing.

I slightly think the issue might be in the geo.provider.testing testing harness or mochitest setup because I'd also expect the junit geolocation tests to have issues too in Fission if it was a deeper issue.

Comment 3

[Makoto Kato [:m_kato]]

I guess that this issue is mochitest framework.

Most mochitests for geo location API are failure with fission+xorigin since iframe doesn't have allow attribute.

Comment 4

[Makoto Kato [:m_kato]]

Attachment: Bug 1855518 - Add allow attribute to allow geolocation API with xorigin tests. r=ahal

When running fission+xorigin mochitests, all mochitests for GeoLocation
API are failed. We should allow this API in mochitest's iframe like
fullscreen API.

Comment 5

[Makoto Kato [:m_kato]]

Attachment: Bug 1855518 - Use different URL for GeoLocation xorigin test. r=#dom-core

When running fission+xorigin mochitest, URL will be example.org.
Since this is cross origin test, we should use another URL.

Depends on D193752

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ccebd8424590
Add allow attribute to allow geolocation API with xorigin tests. r=ahal
https://hg.mozilla.org/integration/autoland/rev/40164cef1755
Use different URL for GeoLocation xorigin test. r=dom-core,mccr8

Comment 7

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/ccebd8424590
https://hg.mozilla.org/mozilla-central/rev/40164cef1755