[webkit-dev] Please don't land patches without rebaselining tests for at least one platform

[Tim Horton]

On Mar 22, 2013, at 1:34 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:

> On Thu, Mar 21, 2013 at 6:43 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
> On Thu, Mar 21, 2013 at 6:39 PM, Silvia Pfeiffer <silviapf at chromium.org> wrote:
> On Fri, Mar 22, 2013 at 12:30 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
> On Thu, Mar 21, 2013 at 6:05 PM, Maciej Stachowiak <mjs at apple.com> wrote:
> On Mar 21, 2013, at 5:38 PM, Glenn Adams <glenn at skynav.com> wrote:
>> On Thu, Mar 21, 2013 at 6:11 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
>> On Thu, Mar 21, 2013 at 5:10 PM, Glenn Adams <glenn at skynav.com> wrote:
>> 
>> On Thu, Mar 21, 2013 at 5:55 PM, Ryosuke Niwa <rniwa at webkit.org> wrote:
>> On Thu, Mar 21, 2013 at 4:50 PM, Glenn Adams <glenn at skynav.com> wrote:
>> That's my platform, so I have to manage with it.
>> 
>> I do have a Retina MBP too but I don't use it to work on the rendering engine precisely because of this issue.  It's expected that every contributor has access to a machine where he/she can run layout tests.  Retina MBP is not such a machine.
>> 
>> Well, it's been working for me.
>> 
>> The fact you appears to be contributing patches without appropriate rebaselines seems to indicate that it's not working for us.
>> 
>> Oh, please point out a case of "without appropriate rebaseline". Please point out in the documentation where "appropriate rebaseline" is defined. I think you are making unwarranted assumptions here. If you can't define or understand a process where I can contribute using a MBP Retina, then I think you are imposing an arbitrary, unwarranted restriction on the community. I have been contributing successfully, ergo, it is working.
>> 
>> Many are contributing WebCore layout and rendering patches using a wide variety of platforms, not all of which match your platform assumptions. It is not reasonable to claim they aren't contributing positively or that their contributions don't work.
> 
> We should definitely make it possible to contribute using a Retina system. Apple's flagship laptops offer Retina displays, and it would be crazy to rule them out as development machines. I'd imagine one day we may want the canonical Mac pixel results to be *only* retina.
> 
> Yes, we should but it isn't today.
> 
> Perhaps one possibility is to make it possible to generate non-Retina pixel results on a Retina system. That seems eminently doable to me, unless there's something I am missing.
> 
> Yeah, Alexey and I were talking about this earlier. We need a some way to force CAGraphics, etc… to behave as if we're in non-Retina MBP. We definitely don't want to check in Retina pixel results.
> 
> Where can I sign up to make this a higher priority. ;-)
> 
> Post a patch on https://bugs.webkit.org/show_bug.cgi?id=93673.
> 
> Tim (thorton) kindly took time to fix this problem in http://trac.webkit.org/changeset/146650 at least for render tree dumps. Pixel tests still do fail for obvious reasons but this is a huge improvement nonetheless.

Obvious reasons == "they fail on 1x machines too", right?

> Thanks Tim!

Sure!

> - R. Niwa
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20130322/0bb26764/attachment.html>