Make run_webkit_tests work on Windows 7. ...

Make run_webkit_tests work on Windows 7. 

We modify test_shell --check-sys-deps to allow Win 7, add a "chromium-win-7"
port to platform_utils, a "WIN-7" port to test_expectations, and create
an empty shell directory under webkit/data/layout_tests to make the
regression scripts happy.

The actual new baselines will follow in a separate CL.

  R=ojan@chromium.org
  TEST=none
  BUG=none


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

[ojan, 2009-10-29 20:15:14]

http://codereview.chromium.org/342033/diff/6001/6004
File webkit/tools/test_shell/test_shell_platform_delegate_win.cc (right):

http://codereview.chromium.org/342033/diff/6001/6004#newcode64
Line 64: isWin7 = true;
This is unused. Don't we want to send WIN7 down the isVista path below?

[ojan, 2009-10-29 20:32:31]

LGTM

http://codereview.chromium.org/342033/diff/6006/7003
File webkit/tools/test_shell/test_shell_platform_delegate_win.cc (right):

http://codereview.chromium.org/342033/diff/6006/7003#newcode73
Line 73: ; // XP
Nit: This seems weird to me. That compiles?

How about instead checking for the following and adding the error in that case?

!(osvi.dwMajorVersion == 5
  && osvi.dwMinorVersion == 1
  && osvi.wProductType == VER_NT_WORKSTATION)

[Dirk Pranke, 2009-10-29 21:03:19]

http://codereview.chromium.org/342033/diff/6001/6004
File webkit/tools/test_shell/test_shell_platform_delegate_win.cc (right):

http://codereview.chromium.org/342033/diff/6001/6004#newcode64
Line 64: isWin7 = true;
On 2009/10/29 20:15:14, ojan wrote:
> This is unused. Don't we want to send WIN7 down the isVista path below?

Good catch. Fixed.

http://codereview.chromium.org/342033/diff/6006/7003#newcode73
Line 73: && osvi.wProductType == VER_NT_WORKSTATION) {
On 2009/10/29 20:32:31, ojan wrote:
> Nit: This seems weird to me. That compiles?
> 
> How about instead checking for the following and adding the error in that
case?
> 
> !(osvi.dwMajorVersion == 5
>   && osvi.dwMinorVersion == 1
>   && osvi.wProductType == VER_NT_WORKSTATION)
> 
> 

Sure, it compiles - it's just an empty statement. But your suggestion is better.