Closed Bug 1354006 Opened 5 years ago Closed 5 years ago

[selectors4] Remove "interact" flag from :focus-within tests


(Core :: CSS Parsing and Computation, enhancement)

Not set



Tracking Status
firefox55 --- fixed


(Reporter: rego, Assigned: rego)



(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36

Steps to reproduce:

This comes from a PR in WPT repo:

The issue is that some reference tests are marked with "interact" flag,
which causes them to be considered manual tests instead of automatic.
However they are actually run automatically with some JavaScript.

As we're cannot modify the tests in WPT repo under css/vendor-imports/,
this should be done directly on Firefox source code.
This patch has been extracted from:

The problem is that the "interact" flag on these tests
causes that they are considered manual,
when they can be run automatically.

This patch removes the "interact" flag and uses "reftest-wait" class
to be sure that the test has been completed.
The tests were originally introduced in bug #1176997.

This is my first attempt to upload a patch here, so please forgive me if I did something wrong. :-)
Component: Untriaged → CSS Parsing and Computation
Product: Firefox → Core
Comment on attachment 8855199 [details] [diff] [review]
[selectors4] Remove "interact" flag from :focus-within tests

r?xidorn since he reviewed the layout parts in bug 1176997
Attachment #8855199 - Flags: review?(xidorn+moz)
Comment on attachment 8855199 [details] [diff] [review]
[selectors4] Remove "interact" flag from :focus-within tests

Review of attachment 8855199 [details] [diff] [review]:

Looks reasonable.
Attachment #8855199 - Flags: review?(xidorn+moz) → review+
Assignee: nobody → rego
Keywords: checkin-needed
Pushed by
[selectors4] Remove "interact" flag from :focus-within tests. r=xidorn
Keywords: checkin-needed
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.