[Piglit] default piglit window size change

[Anuj Phogat]

On Fri, Nov 2, 2012 at 9:32 AM, Brian Paul <brianp at vmware.com> wrote:
>
> I think I've brought this up in the past.  On Windows, the minimum width of
> a window is 116 pixels (at least in my current Win7 setup). A lot of piglit
> tests create 100x100 pixels and when piglit creates the window, Win7 resizes
> it to 116x100.  The framework code detects this change and reports:
>
> Got spurious window resize in automatic run (100,100 to 116,100)
> PIGLIT: {'result': 'warn'}
>
> So a bunch of tests get the 'warn' result instead of being run properly.  I
> think this changed at some point during the recent framework/waffle
> overhaul.
>
> My proposal for this is to set a default window size of 150x150 in the
> piglit-framework-gl.h header:
>
> --- a/tests/util/piglit-framework-gl.h
> +++ b/tests/util/piglit-framework-gl.h
> @@ -216,6 +216,13 @@ piglit_gl_test_run(int argc, char *argv[],
>
>          \
>                  config.init = piglit_init;          \
>                  config.display = piglit_display;          \
> +         \
> +                /* Default window size.  Note: Win7's min window width */
> \
> +               /* seems to be 116 pixels.  When the window size is */
> \
> +                /* unexpectedly resized, tests are marked as "WARN". */
> \
> +                /* Let's use a larger default to avoid that. */         \
> +                config.window_width = 150;         \
> +                config.window_height = 150;
>
>  #define PIGLIT_GL_TEST_CONFIG_END          \
>
>
> And then remove these config lines from a bunch of tests:
>
>        config.window_width = 100;
>        config.window_height = 100;
>
> Of course, I'll have to inspect these tests to make sure they don't rely on
> that specific window size, but I bet few if any of them do.
>
> This change fixes things here.
>
> Sound OK?
>
> -Brian
>

Yes, This looks reasonable. Now piglit test need not set width/height unless
specific dimensions are required.