-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a REFERENCE-IN-OTHER-TYPE lint #46120
Conversation
@@ -2,7 +2,6 @@ | |||
<link rel="author" title="Morten Stenshorne" href="mailto:mstensho@chromium.org"> | |||
<link rel="help" href="https://www.w3.org/TR/css-break-3/#transforms"> | |||
<link rel="help" href="https://www.w3.org/TR/CSS22/visuren.html#relative-positioning"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like this test would benefit from a "testharness is a child of <head>
" lint :p
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are in the head. An explicit head tag is optional in HTML.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the manifest these are all testharness tests, but WebKit's historic slightly different logic has led to these being run in a bogus state previously (where they were treated as both a reftest and a testharness test all at once).
It's not overly meaningful to have a reference while also being manual, and all this is really testing is that the click event is fired, so let's just turn this into a testharness test using testdriver.
We already check whether tests crash, so we may as well check the rendering too.
This is similar to the TESTHARNESS-IN-OTHER-TYPE lint
Fixes #46058, and fixes a bunch of the failures it catches.