Closed Bug 1666275 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 25649 - sensors: Only call test_driver.set_permission from the top-level window.

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox83 fixed)

RESOLVED FIXED
83 Branch
Tracking Status
firefox83 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 25649 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/25649
Details from upstream follow.

b'Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>' wrote:

sensors: Only call test_driver.set_permission from the top-level window.

testdriver.js' documentation says "presently, testdriver.js only works
in the top-level test browsing context (and not therefore in any frame
or window opened from it".

If we do not restrict the set_permission() call here, the call to
initialize_generic_sensor_tests() in
generic-sensor/resources/iframe_sensor_handler.html will time out
waiting for it to complete when running the iframe tests outside the
Chromium CI (i.e. in wpt.fyi or with "wpt run").

Change-Id: I522a5ebacb7da9e35bbe2ea3564d3288a34c653b

Reviewed-on: https://chromium-review.googlesource.com/2421671
WPT-Export-Revision: f4e7c9e372729cfad071d4350a1a722bec228243

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2a4b6db38672
[wpt PR 25649] - sensors: Only call test_driver.set_permission from the top-level window., a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.