Sputnik.py

This patch allow the launcher to run the code in both
strict and no strict mode.

When the launcher run in non strict mode the launcher put at the top of the test "var strict_mode = false" but if it runs in strict mode then it inject 
"use strict"
var strict_mode = true.

The runner runs each test case twice. Once in strict mode and once in non strict mode. 

There is also a new metadata that can be add to the test @strict_mode_negative, this metadata is similar to the current @negative, but this one will indicate that the test will fail only in strict mode.

[christian.plesner.hansen, 2010-08-03 23:32:26]

Looks good.  See comments.

[christian.plesner.hansen, 2010-08-03 23:35:32]

Arg, publishing comments doesn't work.  Here are the comments manually.

> print "%s was expected to fail in strict mode, but didn't" %name
Space between % and name

> elif self.case.strict_mode and self.case.IsStrictModeNegative():
I would change this to

elif self.case.strict_mode:
  if self.case.IsStrictModeNegative():
    ... // failed as expected
  else:
    ... // passed

> case.strict_mode = True
Rather than execute the same case two times I would suggest creating two test
cases for each test, one strict and one non-strict.

[pedro.delgallego, 2010-08-04 11:01:50]

Two test cases are created (one strict and one non-strict) instead of executing
two times the same.

[pedro.delgallego, 2010-08-04 11:04:03]

> Arg, publishing comments doesn't work.  Here are the comments manually.

I applied the changes in the new patch. I am not sure if adding  another patch
set in the same issue is the correct option or I should create a new issue?

[christian.plesner.hansen, 2010-08-04 11:59:32]

Please add a patch so we have the full history of the change under one issue.

[christian.plesner.hansen, 2010-08-08 01:44:56]

Please add a command-line option that controls whether strict-mode test cases
are used, with a default value of False so the default behavior of the script is
the same as before this change.  I'll coordinate with the v8 team and make sure
they're ready before we enable strict mode tests by default.

Feel free to submit after adding the flag without waiting for another round of
review for this CL.