Put 'this' in a handle in EnsureSize to avoid crash caused by GC at

unlucky moment.
Review URL: http://codereview.chromium.org/40108

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@1419 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/objects.cc b/src/objects.cc
index 32140462e92df0ddab1143690818f7aacfdb8b06..5e6a2dbfebd53b2177bda6d32b0298ca19d9c2aa 100644 (file)
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -4883,6 +4883,7 @@ Object* JSArray::Initialize(int capacity) {
 
 
 void JSArray::EnsureSize(int required_size) {
+  Handle<JSArray> self(this);
   ASSERT(HasFastElements());
   if (elements()->length() >= required_size) return;
   Handle<FixedArray> old_backing(elements());
@@ -4891,8 +4892,9 @@ void JSArray::EnsureSize(int required_size) {
   // constantly growing.
   int new_size = required_size + (required_size >> 3);
   Handle<FixedArray> new_backing = Factory::NewFixedArray(new_size);
+  // Can't use this any more now because we may have had a GC!
   for (int i = 0; i < old_size; i++) new_backing->set(i, old_backing->get(i));
-  SetContent(*new_backing);
+  self->SetContent(*new_backing);
 }