LGTM. Nit in the description: "BUG=129926" in a separate line would look nicer.
https://chromiumcodereview.appspot.com/10451092/diff/1/src/jsregexp.cc File src/jsregexp.cc (right): https://chromiumcodereview.appspot.com/10451092/diff/1/src/jsregexp.cc#newcode5617 src/jsregexp.cc:5617: ZoneList<GuardedAlternative>* alts = alternatives(); We don't check recursion_depth here, because nesting of choice nodes is limited somewhere else? https://chromiumcodereview.appspot.com/10451092/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
