Skip to content

Commit

Permalink
Bug 1583621 [wpt PR 19267] - Fix xrWebGLLayer_constructor test, a=tes…
Browse files Browse the repository at this point in the history
…tonly

Automatic update from web-platform-tests
Fix xrWebGLLayer_constructor test

xrWebGLLayer_constructor was mistakenly updated to require user
activation though the spec does not explicitly require it in this case
since the inline session does not request any features.  Even were it
to require this, the way that it was structured caused there to not be
a promise set when the ".then" was caused, giving a script error.

This change essentially reverts WPT Commit 16d4388, and was discussed
with Mozilla when deciding which way to fix it.

Bug: 1007304
Change-Id: I824f63cb172aed5928c55224b31c00803b5ddaa2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1822821
Commit-Queue: Jacob DeWitt <jacdechromium.org>
Commit-Queue: Alexander Cooper <alcooperchromium.org>
Auto-Submit: Alexander Cooper <alcooperchromium.org>
Reviewed-by: Jacob DeWitt <jacdechromium.org>
Cr-Commit-Position: refs/heads/master{#699467}

--

wpt-commits: 75515ddfc56f2228ea9444f61f24cd78eba16aa6
wpt-pr: 19267

UltraBlame original commit: 5e39bf59dbf6b20075aafa9b97e7caab251cc369
  • Loading branch information
marco-c committed Oct 5, 2019
1 parent 455e134 commit 163e70c
Showing 1 changed file with 1 addition and 23 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -157,23 +157,7 @@
=
>
{
let
sessionPromise
;
navigator
.
xr
.
test
.
simulateUserActivation
(
function
(
)
{
sessionPromise
=
return
navigator
.
xr
Expand All @@ -184,12 +168,6 @@
inline
'
)
;
}
)
;
return
sessionPromise
.
then
(
Expand Down

0 comments on commit 163e70c

Please sign in to comment.