Closed Bug 1723947 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 29891 - Fix WPT linter error

Categories

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

task

Tracking

(firefox92 fixed)

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Tom McKee <tommckee@chromium.org>' wrote:

Fix WPT linter error

In https://crrev.com/c/2961075 we migrated away from using
wpt/resource-timing/resources/all_resource_types.html but didn't remove the
file. We also updated wpt/lint.ignore to remove mention of
all_resource_types.html which caused downstream CI to fail
(https://github.com/web-platform-tests/wpt/pull/29786#issuecomment-892184957).

This change removes the unused file which, apart from cleaning up after
ourselves, should fix the downstream linter.

I've filed crbug.com/1236498 to follow up on getting chromium's presubmit to
keep us from making this mistake again.

Bug: 1236498
Change-Id: Id0556b08643b9c1994ce06877e4be6fb6e791fc4
Reviewed-on: https://chromium-review.googlesource.com/3071040
WPT-Export-Revision: c54d6f4ca5c2a57975b95379090d11977226c39f

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/05605ffa45c9 [wpt PR 29891] - Fix WPT linter error, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in before you can comment on or make changes to this bug.