Re: Changing reftest required resolution

2012-08-29 Thread jmaher
On Wednesday, August 29, 2012 11:35:52 AM UTC-4, Andrew Halberstadt wrote: > On 08/29/2012 09:56 AM, Andrew Halberstadt wrote: > > > On 08/28/2012 02:17 PM, L. David Baron wrote: > > >> On Tuesday 2012-08-28 12:52 -0400, Andrew Halberstadt wrote: > > >> I also don't think we should go quite as s

Re: Changing reftest required resolution

2012-08-29 Thread Andrew Halberstadt
On 08/29/2012 09:56 AM, Andrew Halberstadt wrote: On 08/28/2012 02:17 PM, L. David Baron wrote: On Tuesday 2012-08-28 12:52 -0400, Andrew Halberstadt wrote: I also don't think we should go quite as small as 400x400 -- and we want to come up with a common value with other browser vendors that are

Re: Changing reftest required resolution

2012-08-29 Thread Andrew Halberstadt
On 08/28/2012 02:17 PM, L. David Baron wrote: On Tuesday 2012-08-28 12:52 -0400, Andrew Halberstadt wrote: I also don't think we should go quite as small as 400x400 -- and we want to come up with a common value with other browser vendors that are also using reftest. We don't want to be running o

Re: Changing reftest required resolution

2012-08-28 Thread L. David Baron
On Wednesday 2012-08-29 17:08 +1200, Robert O'Callahan wrote: > On Wed, Aug 29, 2012 at 6:17 AM, L. David Baron wrote: > > > We don't want to be running our reftests at > > a size smaller than the accepted max size for reftests at W3C. > > > > What is the current required size for W3C reftests?

Re: Changing reftest required resolution

2012-08-28 Thread Robert O'Callahan
On Wed, Aug 29, 2012 at 6:17 AM, L. David Baron wrote: > We don't want to be running our reftests at > a size smaller than the accepted max size for reftests at W3C. > What is the current required size for W3C reftests? I can't find any documentation of that. Rob -- “You have heard that it was

Re: Changing reftest required resolution

2012-08-28 Thread L. David Baron
On Tuesday 2012-08-28 11:17 -0700, L. David Baron wrote: > On Tuesday 2012-08-28 12:52 -0400, Andrew Halberstadt wrote: > > == The Problem == > > > > Reftests currently assume a window size of 800x1000. This is not > > possible on mobile devices, and using this resolution on tegras and > > pandas

Re: Changing reftest required resolution

2012-08-28 Thread jmaher
On Tuesday, August 28, 2012 2:36:16 PM UTC-4, halb...@gmail.com wrote: > On Tuesday, August 28, 2012 1:58:43 PM UTC-4, Benoit Girard wrote: > > > I've already done this work but we decided to just increase the > > > > > > resolution for our tegra board: > > > > > > > > > > > > See https:

Re: Changing reftest required resolution

2012-08-28 Thread halbersa
On Tuesday, August 28, 2012 1:58:43 PM UTC-4, Benoit Girard wrote: > I've already done this work but we decided to just increase the > > resolution for our tegra board: > > > > See https://bugzilla.mozilla.org/show_bug.cgi?id=66 > > which includes an outdated patch that adds a screen(w,h)

Re: Changing reftest required resolution

2012-08-28 Thread L. David Baron
On Tuesday 2012-08-28 12:52 -0400, Andrew Halberstadt wrote: > == The Problem == > > Reftests currently assume a window size of 800x1000. This is not > possible on mobile devices, and using this resolution on tegras and > pandas consumes too much memory and results in many timeouts and > random-fa

Re: Changing reftest required resolution

2012-08-28 Thread Benoit Girard
I've already done this work but we decided to just increase the resolution for our tegra board: See https://bugzilla.mozilla.org/show_bug.cgi?id=66 which includes an outdated patch that adds a screen(w,h) annotation to each test and a patch to compute the required size per test. On Tue, Aug 2

Re: Changing reftest required resolution

2012-08-28 Thread Jeff Hammel
If the exact width/height could be found for each test then these could be marked in a manifest (theoretically, not speaking to the existing reftest manifest format per se). Then reftest could be modified to take a (e.g.) --resoltuion 400x400 argument and the test runner passing over any test

Changing reftest required resolution

2012-08-28 Thread Andrew Halberstadt
== The Problem == Reftests currently assume a window size of 800x1000. This is not possible on mobile devices, and using this resolution on tegras and pandas consumes too much memory and results in many timeouts and random-failures. Changing the resolution to something like 400x400 is very s