Reviewers: Erik Corry,

Message:
Erik, could you please take a look at this?

I also uploaded a related patch to webkit:
https://bugs.webkit.org/show_bug.cgi?id=81200


https://chromiumcodereview.appspot.com/9701093/diff/1/src/heap.cc
File src/heap.cc (right):

https://chromiumcodereview.appspot.com/9701093/diff/1/src/heap.cc#newcode4824
src/heap.cc:4824: void Heap::AdvanceIdleIncrementalMarking(intptr_t
step_size) {
This function is extracted from lines 4876-4894 of IdleNotification().

https://chromiumcodereview.appspot.com/9701093/diff/1/src/heap.cc#newcode4868
src/heap.cc:4868: if (hint >= 1000 || !FLAG_incremental_marking ||
Once this CL lands into Chromium without memory regressions, I will
refactor this to remove the (hint >= 1000) condition and get rid of
IdleGlobalGC.

Description:
Adjust heuristics for idle notifications after context disposal.


Please review this at https://chromiumcodereview.appspot.com/9701093/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/heap.h
  M src/heap.cc


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to