Closed Bug 1231798 Opened 4 years ago Closed 4 years ago

Intermittent TEST-UNEXPECTED-TIMEOUT | /custom-elements/custom-element-lifecycle/types-of-callbacks/attribute-changed-callback-change-attribute-test.html | expected OK

Categories

(Testing :: web-platform-tests, defect)

defect
Not set

Tracking

(firefox45 fixed, firefox46 fixed)

RESOLVED FIXED
mozilla46
Tracking Status
firefox45 --- fixed
firefox46 --- fixed

People

(Reporter: philor, Assigned: bkelly)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

This seems to be mostly happening since disabling attached-callback-move-element-test.html in https://hg.mozilla.org/integration/mozilla-inbound/rev/8dd58892051a

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=platform%20%281%29%20opt%20linux&tochange=707a87454058&fromchange=a1caa7e7949f&selectedJob=18447654

I suspect we've just moved the intermittent from one test to another.
After talking with James and William, I am going to disable all the custom-element wpt tests.  We have an impl, but its behind a pref.  I will file a follow-up bug to investigate the tests and re-enable.
Blocks: 1234886
Assignee: nobody → bkelly
Status: NEW → ASSIGNED
Flags: needinfo?(james)
Attachment #8701560 - Flags: review?(james)
Comment on attachment 8701560 [details] [diff] [review]
Mass disable custom-elements wpt tests since they are not actually testing anything currently. r=jgraham

Review of attachment 8701560 [details] [diff] [review]:
-----------------------------------------------------------------

But there is an easier way; it should also work to just put a __dir__.ini under /custom-elements/ and I would have left all the metadata files in as a reference for when we reenable the tests.
Attachment #8701560 - Flags: review?(james) → review+
(In reply to James Graham [:jgraham] from comment #8)
> But there is an easier way; it should also work to just put a __dir__.ini
> under /custom-elements/ and I would have left all the metadata files in as a
> reference for when we reenable the tests.

I've seen the recursion of __dir__.ini not work on some platforms.  Maybe thats fixed, but I didn't have time to try a lot of different things today.

And I removed the other metadata because it should all be different with the dom.webcomponents.enabled pref turned on.  All new metadata will have to be figured out.

Thanks for the review.  Have a great holiday.
https://hg.mozilla.org/mozilla-central/rev/200ab9a3b18a
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.