When allocation is forced because we already did two GCs we need to force GCs...

src/api.cc

 // Copyright 2009 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 89 matching lines @@
 // --- E x c e p t i o n   B e h a v i o r ---
 
 
 static FatalErrorCallback exception_behavior = NULL;
 int i::Internals::kJSObjectType = JS_OBJECT_TYPE;
 int i::Internals::kFirstNonstringType = FIRST_NONSTRING_TYPE;
 int i::Internals::kProxyType = PROXY_TYPE;
 
 static void DefaultFatalErrorHandler(const char* location,
                                      const char* message) {
+  printf("%s: %s\n", location, message);

[antonm, 2009/10/02 12:53:33]

is it a reminder of debugging or intended change?

[Erik Corry, 2009/10/02 13:09:24]

I guess it's a remainder of debugging.  It certainly would have saved me a lot
of time if it had been in there when I started investigating this issue.  For
some reason the VM crashes shortly after going though this function, so the
message is lost.

I'm going to change it to a PrintF that is only done in DEBUG mode.  This
routine is only called if no other error handler has been registered, so that
seems harmless and potentially very useful.

   ENTER_V8;
   API_Fatal(location, message);
 }
 
 
 
 static FatalErrorCallback& GetFatalErrorHandler() {
   if (exception_behavior == NULL) {
     exception_behavior = DefaultFatalErrorHandler;
   }
@@ skipping 3644 matching lines @@
 
 
 char* HandleScopeImplementer::Iterate(ObjectVisitor* v, char* storage) {
   HandleScopeImplementer* thread_local =
       reinterpret_cast<HandleScopeImplementer*>(storage);
   thread_local->IterateThis(v);
   return storage + ArchiveSpacePerThread();
 }
 
 } }  // namespace v8::internal