[regexp] support \p in character classes.

[regexp] support \p in character classes.

R=littledan@chromium.org
BUG=v8:4743
LOG=N

Committed: https://crrev.com/aba76874dbf3f00741169ef763cdafe4d3600605
Cr-Commit-Position: refs/heads/master@{#34695}

[Yang, 2016-03-09 12:40:51]

Dan, please take a look.

[Yang, 2016-03-09 13:04:36]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-09 13:04:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1774623005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1774623005/20001

[commit-bot: I haz the power, 2016-03-09 13:24:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-09 13:24:10]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-03-10 19:21:37]

On 2016/03/09 13:24:10, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

ping

[Dan Ehrenberg, 2016-03-10 19:35:42]

lgtm

[Yang, 2016-03-10 19:43:12]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-03-10 19:43:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1774623005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1774623005/20001

[commit-bot: I haz the power, 2016-03-10 19:44:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-10 19:44:29]

Try jobs failed on following builders:
  v8_android_arm_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_asan_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel/builds/...)
  v8_linux64_avx2_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel/builds/...)
  v8_linux64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/2740)
  v8_linux_arm64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel/builds/1...)
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/15153)
  v8_linux_dbg_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/2748)
  v8_linux_gcc_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/12215)

[Yang, 2016-03-10 20:09:16]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-03-10 20:09:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
littledan@chromium.org
Link to the patchset: https://codereview.chromium.org/1774623005/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-03-10 20:09:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1774623005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1774623005/40001

[commit-bot: I haz the power, 2016-03-10 23:23:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-10 23:24:28]

Description was changed from

==========
[regexp] support \p in character classes.

R=littledan@chromium.org
BUG=v8:4743
LOG=N
==========

to

==========
[regexp] support \p in character classes.

R=littledan@chromium.org
BUG=v8:4743
LOG=N

Committed: https://crrev.com/aba76874dbf3f00741169ef763cdafe4d3600605
Cr-Commit-Position: refs/heads/master@{#34695}
==========

[commit-bot: I haz the power, 2016-03-10 23:24:29]

Patchset 3 (id:??) landed as
https://crrev.com/aba76874dbf3f00741169ef763cdafe4d3600605
Cr-Commit-Position: refs/heads/master@{#34695}

[PhistucK, 2016-03-26 09:51:08]

phistuck@gmail.com changed reviewers:
+ phistuck@gmail.com

[PhistucK, 2016-03-26 09:51:09]

Why is it sometimes "property class" and sometimes "class property"?
-ZoneList<CharacterRange>* RegExpParser::ParsePropertyClass() {
+bool RegExpParser::ParsePropertyClass(ZoneList<CharacterRange>* result) {
...
+bool RegExpParser::ParseClassProperty(ZoneList<CharacterRange>* ranges) {

[Yang, 2016-03-30 05:04:08]

On 2016/03/26 09:51:09, PhistucK wrote:
> Why is it sometimes "property class" and sometimes "class property"?
> -ZoneList<CharacterRange>* RegExpParser::ParsePropertyClass() {
> +bool RegExpParser::ParsePropertyClass(ZoneList<CharacterRange>* result) {
> ...
> +bool RegExpParser::ParseClassProperty(ZoneList<CharacterRange>* ranges) {

I realize that the naming might a bit confusing.
ParsePropertyClass is parsing a property class specified by \p.
ParseClassProperty is parsing a property class inside a character class, like
[\pL]. This is symmetrical to similar functions such as ParseClassAtom or
ParseClassCharacterEscape.