[regexp][liveedit] Fix inconsistent JSArrays

src/js/regexp.js

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 (function(global, utils) {
 
 %CheckIsBootstrapping();
 
 // -------------------------------------------------------------------
 // Imports
@@ skipping 20 matching lines @@
   MinSimple = from.MinSimple;
   SpeciesConstructor = from.SpeciesConstructor;
 });
 
 // -------------------------------------------------------------------
 
 // Property of the builtins object for recording the result of the last
 // regexp match.  The property RegExpLastMatchInfo includes the matchIndices
 // array of the last successful regexp match (an array of start/end index
 // pairs for the match and all the captured substrings), the invariant is
-// that there are at least two capture indeces.  The array also contains
+// that there are at least two capture indices.  The array also contains
 // the subject string for the last successful match.
-var RegExpLastMatchInfo = new InternalPackedArray(
- 2,                 // REGEXP_NUMBER_OF_CAPTURES
- "",                // Last subject.
- UNDEFINED,         // Last input - settable with RegExpSetInput.
- 0,                 // REGEXP_FIRST_CAPTURE + 0
- 0                  // REGEXP_FIRST_CAPTURE + 1
-);
+// We use a JSObject rather than a JSArray so we don't have to manually update
+// its length.
+var RegExpLastMatchInfo = {
+  REGEXP_NUMBER_OF_CAPTURES: 2,
+  REGEXP_LAST_SUBJECT:       "",
+  REGEXP_LAST_INPUT:         UNDEFINED,  // Settable with RegExpSetInput.
+  CAPTURE0:                  0,
+  CAPTURE1:                  0
+};
 
 // -------------------------------------------------------------------
 
 // ES#sec-isregexp IsRegExp ( argument )
 function IsRegExp(o) {
   if (!IS_RECEIVER(o)) return false;
   var is_regexp = o[matchSymbol];
   if (!IS_UNDEFINED(is_regexp)) return TO_BOOLEAN(is_regexp);
   return IS_REGEXP(o);
 }
@@ skipping 1134 matching lines @@
 
 for (var i = 1; i < 10; ++i) {
   utils.InstallGetterSetter(GlobalRegExp, '$' + i, RegExpMakeCaptureGetter(i),
                             NoOpSetter, DONT_DELETE);
 }
 %ToFastProperties(GlobalRegExp);
 
 // -------------------------------------------------------------------
 // Internal
 
-var InternalRegExpMatchInfo = new InternalPackedArray(2, "", UNDEFINED, 0, 0);
+var InternalRegExpMatchInfo = {
+  REGEXP_NUMBER_OF_CAPTURES: 2,
+  REGEXP_LAST_SUBJECT:       "",
+  REGEXP_LAST_INPUT:         UNDEFINED,
+  CAPTURE0:                  0,
+  CAPTURE1:                  0
+};
 
 function InternalRegExpMatch(regexp, subject) {
   var matchInfo = %_RegExpExec(regexp, subject, 0, InternalRegExpMatchInfo);
   if (!IS_NULL(matchInfo)) {
     RETURN_NEW_RESULT_FROM_MATCH_INFO(matchInfo, subject);
   }
   return null;
 }
 
 function InternalRegExpReplace(regexp, subject, replacement) {
   return %StringReplaceGlobalRegExpWithString(
       subject, regexp, replacement, InternalRegExpMatchInfo);
 }
 
 // -------------------------------------------------------------------
 // Exports
 
 utils.Export(function(to) {
   to.InternalRegExpMatch = InternalRegExpMatch;
   to.InternalRegExpReplace = InternalRegExpReplace;
   to.IsRegExp = IsRegExp;
   to.RegExpExec = DoRegExpExec;
   to.RegExpInitialize = RegExpInitialize;
   to.RegExpLastMatchInfo = RegExpLastMatchInfo;
   to.RegExpTest = RegExpTest;
 });
 
 })