Summary: | Re-sync W3C web-platform-tests' html/dom/interfaces.html | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, commit-queue, darin, rniwa, sam | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Chris Dumez
2016-02-19 10:26:43 PST
Created attachment 271764 [details]
Patch
Comment on attachment 271764 [details]
Patch
What about iOS test results?
(In reply to comment #2) > Comment on attachment 271764 [details] > Patch > > What about iOS test results? Are developers really expected to rebaseline tests for all platforms? It seems easier to rebaseline after landing, using the actual results from the bot than to generate the baseline locally for other platforms. For a change like this, it seems easier to update the results at least for iOS, as supposedly they are exactly the same. We do want to run tests on iOS EWS, which will obviously make it a requirement. I personally still think that this particular test should be simply deleted, maintaining it is a daily annoyance. Created attachment 271765 [details]
Patch
(In reply to comment #4) > For a change like this, it seems easier to update the results at least for > iOS, as supposedly they are exactly the same. > > We do want to run tests on iOS EWS, which will obviously make it a > requirement. > > I personally still think that this particular test should be simply deleted, > maintaining it is a daily annoyance. Only because it is being actively worked on and we are making daily progress on it. I don't see it as being a long term issue as our pass-rate on this test is becoming really good. This test certainly has been very useful to me. Comment on attachment 271765 [details] Patch Clearing flags on attachment: 271765 Committed r196830: <http://trac.webkit.org/changeset/196830> All reviewed patches have been landed. Closing bug. |