Closed Bug 1526619 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 15283 - [Payment Request][WPT] Fix abort() tests.

Categories

(Core :: DOM: Web Payments, enhancement, P4)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Rouslan Solomakhin <rouslan@chromium.org> wrote:

[Payment Request][WPT] Fix abort() tests.

Before this patch, the WPT tests for PaymentRequest.abort() were
expected to fail due to including the vendor testdriver JavaScript file
before the generic testdriver file. The vendor file was attempting to
use the objects defined in the generic file, but the incorrect include
order caused a JS error.

This patch reorders the vendor JS file to be after the generic JS file
and splits out the manual tests that require a user gesture for
PaymentRequest.show().

After this patch, the PaymentRequest.abort() tests are passing.

Bug: 929773
Change-Id: Ic0f8ff84b478116821e98775d1cec31cfad32d9f
Reviewed-on: https://chromium-review.googlesource.com/1459588
WPT-Export-Revision: 84300acf0a0e1f90e315fea91e25e8840bf66f6e

Whiteboard: [wptsync downstream] → [wptsync downstream error]
Component: web-platform-tests → DOM: Web Payments
Product: Testing → Core
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Ran 1 tests and 2 subtests
OK     : 1
PASS   : 2
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/55f8f5e09d25
[wpt PR 15283] - [Payment Request][WPT] Fix abort() tests., a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/98162bdbcabb
[wpt PR 15283] - [Payment Request][WPT] Fix abort() tests., a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.