Fix RegExp white-space character class to match BOMs.

Fix RegExp white-space character class to match BOMs.

R=rossberg@chromium.org
TEST=test262/S15.10.2.12_A?_T1,mjsunit/regexp

Committed: https://code.google.com/p/v8/source/detail?r=10770

[Michael Starzinger, 2012-02-20 12:10:07]

This is consistent with recent JSC, but recent SM still is non-ES5 in that
regard.

[Lasse Reichstein Nielsen, 2012-02-20 13:55:30]

https://chromiumcodereview.appspot.com/9426032/diff/1/test/sputnik/sputnik.st...
File test/sputnik/sputnik.status (right):

https://chromiumcodereview.appspot.com/9426032/diff/1/test/sputnik/sputnik.st...
test/sputnik/sputnik.status:134: S15.10.2.12_A2_T1: PASS || FAIL_OK
If you can, make it either PASS or FAIL_OK.
This way, all you test is whether it crashes.

[rossberg, 2012-02-20 16:03:19]

http://codereview.chromium.org/9426032/diff/1/src/jsregexp.cc
File src/jsregexp.cc (right):

http://codereview.chromium.org/9426032/diff/1/src/jsregexp.cc#newcode3600
src/jsregexp.cc:3600: static const int kSpaceRangeCount = 22;
Not your code, but can't we compute this constant using ARRAY_SIZE?

[Michael Starzinger, 2012-02-20 16:34:00]

PTAL. Added new patch set.

https://chromiumcodereview.appspot.com/9426032/diff/1/src/jsregexp.cc
File src/jsregexp.cc (right):

https://chromiumcodereview.appspot.com/9426032/diff/1/src/jsregexp.cc#newcode...
src/jsregexp.cc:3600: static const int kSpaceRangeCount = 22;
On 2012/02/20 16:03:19, rossberg wrote:
> Not your code, but can't we compute this constant using ARRAY_SIZE?

Done.

https://chromiumcodereview.appspot.com/9426032/diff/1/test/sputnik/sputnik.st...
File test/sputnik/sputnik.status (right):

https://chromiumcodereview.appspot.com/9426032/diff/1/test/sputnik/sputnik.st...
test/sputnik/sputnik.status:134: S15.10.2.12_A2_T1: PASS || FAIL_OK
On 2012/02/20 13:55:30, Lasse Reichstein Nielsen wrote:
> If you can, make it either PASS or FAIL_OK.
> This way, all you test is whether it crashes.

Done. Sputnik will be deprecated (and replaced by test262) this quarter anyways,
so I don't actually care much about keeping this file clean.

[rossberg, 2012-02-20 16:52:43]

lgtm