Fix corner-case in heap size estimation.

src/mark-compact.cc

 // Copyright 2011 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 3615 matching lines @@
       continue;
     }
 
     // One unused page is kept, all further are released before sweeping them.
     if (p->LiveBytes() == 0) {
       if (unused_page_present) {
         if (FLAG_gc_verbose) {
           PrintF("Sweeping 0x%" V8PRIxPTR " released page.\n",
                  reinterpret_cast<intptr_t>(p));
         }
+        space->MarkPageForLazySweeping(p);

[Vyacheslav Egorov (Chromium), 2012/01/17 13:49:37]

this is pretty confusing. we are both marking it for lazy sweepign and releasing
it immediately.

maybe rename MarkPageForLazySweeping into something else?

[Michael Starzinger, 2012/01/17 14:09:39]

Done. As discussed offline we chose IncreaseUnsweptFreeBytes  (and the Decrease
counterpart) to be a suitable name here.

         space->ReleasePage(p);
         continue;
       }
       unused_page_present = true;
     }
 
     if (lazy_sweeping_active) {
       if (FLAG_gc_verbose) {
         PrintF("Sweeping 0x%" V8PRIxPTR " lazily postponed.\n",
                reinterpret_cast<intptr_t>(p));
@@ skipping 261 matching lines @@
   while (buffer != NULL) {
     SlotsBuffer* next_buffer = buffer->next();
     DeallocateBuffer(buffer);
     buffer = next_buffer;
   }
   *buffer_address = NULL;
 }
 
 
 } }  // namespace v8::internal