Changed MarkedSpace to delegate grow/shrink decisions to Heap
Created attachment 81193 [details] Patch
Committed r77612: <http://trac.webkit.org/changeset/77612>
http://trac.webkit.org/changeset/77612 might have broken SnowLeopard Intel Release (Tests) The following tests are not passing: accessibility/aria-activedescendant-crash.html accessibility/aria-checkbox-checked.html accessibility/aria-checkbox-text.html accessibility/aria-combobox.html accessibility/aria-controls-with-tabs.html
Reopening bug. I see sporadic assertions on both 10.5 and 10.6. An example: ASSERTION FAILED: m_heap.usedBlocks > neededBlocks (/Users/nikolaszimmermann/Coding/WebKit/Source/JavaScriptCore/runtime/MarkedSpace.cpp:174 void JSC::MarkedSpace::shrinkBlocks(size_t)) -> JSC::MarkedSpace::shrinkBlocks(unsigned long) -> JSC::MarkedSpace::sweep() -> JSC::Heap::reset(JSC::Heap::SweepToggle) -> JSC::Heap::collectAllGarbage() -> JSC::DefaultGCActivityCallbackPlatformData::trigger(__CFRunLoopTimer*, void*) -> __CFRunLoopRun -> CFRunLoopRunSpecific -> RunCurrentEventLoopInMode -> ReceiveNextEventCommon -> BlockUntilNextEventMatchingListInMode -> _DPSNextEvent -> -[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:] -> 0x1000162f4 -> -[NSApplication run] -> NSApplicationMain -> 0x10000a1c0 SL bot crashes all tests as well.
rolled out Committed r77619 M Source/JavaScriptCore/runtime/MarkedSpace.cpp M Source/JavaScriptCore/runtime/Heap.cpp M Source/JavaScriptCore/runtime/Heap.h M Source/JavaScriptCore/runtime/MarkedSpace.h M Source/JavaScriptCore/ChangeLog r77619 = c436d330dbed26b8ba04af3f59c77705e7cf3d42 (refs/remotes/trunk)
Created attachment 81301 [details] Patch
Comment on attachment 81301 [details] Patch r=me
Committed r77699: <http://trac.webkit.org/changeset/77699>