One instead of two runtime calls when initializing regexp.

One instead of two runtime calls when initializing regexp.

R=ulan@chromium.org

Committed: https://chromium.googlesource.com/v8/v8/+/bc597301b0504b912a2714b7b3f94d425ae70984

[aandrey, 2014-11-19 13:16:53]

aandrey@chromium.org changed reviewers:
+ aandrey@chromium.org

[aandrey, 2014-11-19 13:16:54]

FYI

https://codereview.chromium.org/719403005/diff/1/src/runtime/runtime-regexp.cc
File src/runtime/runtime-regexp.cc (right):

https://codereview.chromium.org/719403005/diff/1/src/runtime/runtime-regexp.c...
src/runtime/runtime-regexp.cc:803: JSRegExp::Flags
RegExpFlagsFromString(Handle<String> flags, bool* success) {
static?

https://codereview.chromium.org/719403005/diff/1/src/runtime/runtime-regexp.c...
src/runtime/runtime-regexp.cc:808: for (int i = 0; i < flags->length(); i++) {
flags->length() -> length ?

[ulan, 2014-11-19 13:30:19]

LGTM if Andrey's comments are addressed.

[Yang, 2014-11-19 13:45:35]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2014-11-19 13:46:41]

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

[commit-bot: I haz the power, 2014-11-19 14:13:49]

Committed patchset #2 (id:20001)