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

2013-03-23 Thread Silvia Pfeiffer
On Sat, Mar 23, 2013 at 7:34 AM, Ryosuke Niwa rn...@webkit.org wrote:

 On Thu, Mar 21, 2013 at 6:43 PM, Ryosuke Niwa rn...@webkit.org wrote:

 On Thu, Mar 21, 2013 at 6:39 PM, Silvia Pfeiffer 
 silvi...@chromium.orgwrote:

 On Fri, Mar 22, 2013 at 12:30 PM, Ryosuke Niwa rn...@webkit.org wrote:

 On Thu, Mar 21, 2013 at 6:05 PM, Maciej Stachowiak m...@apple.comwrote:

 On Mar 21, 2013, at 5:38 PM, Glenn Adams gl...@skynav.com wrote:

  On Thu, Mar 21, 2013 at 6:11 PM, Ryosuke Niwa rn...@webkit.org
 wrote:

 On Thu, Mar 21, 2013 at 5:10 PM, Glenn Adams gl...@skynav.com
 wrote:


 On Thu, Mar 21, 2013 at 5:55 PM, Ryosuke Niwa rn...@webkit.org
 wrote:

 On Thu, Mar 21, 2013 at 4:50 PM, Glenn Adams gl...@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.


Thanks to both of you! That's awesome!!
Silvia.
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


Re: [webkit-dev] PSA: run-webkit-tests now generates pixel results in retry

2013-03-23 Thread Robert Hogan
 This will allow us to rebaseline BOTH test and image expected results
PRIOR to landing patches.

 You're welcome.


Great stuff. Thanks for doing this!

A question for the Qt/gtk/efl ports: do you plan to put some testers on the
EWS queue? If not, I think you should shout if you do not want authors to
add just the text part of new results and advise what they should do
instead.
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev