Let layout tests run on Windows 7.

Let layout tests run on Windows 7.

TEST=none
BUG=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=30211

[ojan, 2009-10-27 16:01:50]

LGTM. Do all the tests pass?

On Tue, Oct 27, 2009 at 7:37 AM, <maruel@chromium.org> wrote:

> Reviewers: ojan,
>
> Description:
> Let layout tests run on Windows 7.
>
> TEST=none
> BUG=none
>
> Please review this at http://codereview.chromium.org/337043
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     webkit/tools/test_shell/test_shell_platform_delegate_win.cc
>
>
> Index: webkit/tools/test_shell/test_shell_platform_delegate_win.cc
> ===================================================================
> --- webkit/tools/test_shell/test_shell_platform_delegate_win.cc (revision
> 30068)
> +++ webkit/tools/test_shell/test_shell_platform_delegate_win.cc (working
> copy)
> @@ -52,10 +52,11 @@
>   // default to XP metrics, override if on Vista
>   int requiredVScrollSize = 17;
>   int requiredFontSize = -11; // 8 pt
> -  const WCHAR *requiredFont = L"Tahoma";
> +  const wchar_t* requiredFont = L"Tahoma";
> +  // Consider Windows 7 as Vista.
>   bool isVista = false;
>   if (osvi.dwMajorVersion == 6
> -      && osvi.dwMinorVersion == 0
> +      && (osvi.dwMinorVersion == 0 || osvi.dwMinorVersion == 1)
>       && osvi.wProductType == VER_NT_WORKSTATION) {
>     requiredFont = L"Segoe UI";
>     requiredFontSize = -12; // 9 pt
>
>
>

[M-A Ruel, 2009-10-27 16:08:11]

On 2009/10/27 16:01:50, ojan wrote:
> LGTM. Do all the tests pass?

No but I won't be able to fix them if I can't even run them. :)