NEW203068
--new-baseline doesn't take the device type into account
https://bugs.webkit.org/show_bug.cgi?id=203068
Summary --new-baseline doesn't take the device type into account
Simon Fraser (smfr)
Reported 2019-10-16 19:16:34 PDT
Trying to generate a new baseline for iPad: > run-webkit-tests --ipad-simulator fast/scrolling/ios/overflow-scroll-inherited.html --new-baseline < Writing new expected result "platform/ios-simulator-13-wk2/fast/scrolling/ios/overflow-scroll-inherited-expected.txt" Not what I want. It should go into an ipad directory.
Attachments
Jonathan Bedard
Comment 1 2019-10-17 07:27:15 PDT
The trouble with this request (and the reason we didn't do this to begin with) is that it's not totally obvious what behavior you want when generating new baselines. The existing model would indicate that you should actually generate the most specific baseline you possibly can, meaning if you had an 'iPad Pro (9.7-inch)' booted, you would create a baselines for 'iPad Pro (9.7-inch)'. In some cases, (like iPhone 7) that would be correct. In other (your case here, iPad) that would be wrong. My thought here is that we probably need to address <https://bugs.webkit.org/show_bug.cgi?id=203067> first and lean on what was passed via device-type.
Simon Fraser (smfr)
Comment 2 2019-10-17 11:13:26 PDT
Maybe --new-baseline needs arguments?
Simon Fraser (smfr)
Comment 3 2019-10-17 14:10:57 PDT
This is pretty annoying when trying to fix iPad tests. You have to reset results, copy the result to the iPad dir, then git checkout the original result. Yuck.
Note You need to log in before you can comment on or make changes to this bug.