Oilpan: Give a chance to schedule a precise GC after V8 minor and major GCs

Source/platform/heap/ThreadState.cpp

 /*
  * Copyright (C) 2013 Google Inc. 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
@@ skipping 639 matching lines @@
 
 bool ThreadState::shouldSchedulePreciseGC()
 {
 #if ENABLE(IDLE_GC)
     return false;
 #else
     return judgeGCThreshold(1024 * 1024, 1.5);
 #endif
 }
 
+bool ThreadState::shouldScheduleV8FollowupGC()
+{
+    return judgeGCThreshold(32 * 1024 * 1024, 1.5);
+}
+
 bool ThreadState::shouldSchedulePageNavigationGC(float estimatedRemovalRatio)
 {
-    return judgeGCThreshold(1024 * 1024, 1.5);
+    return judgeGCThreshold(1024 * 1024, 1.5 * (1 - estimatedRemovalRatio));

[sof, 2015/08/24 05:12:35]

This change isn't covered by the description; intentionally included?

[haraken, 2015/08/24 05:14:15]

This change has been already landed. Forgot to rebase the CL with ToT.

 }
 
 bool ThreadState::shouldForceConservativeGC()
 {
     // TODO(haraken): 400% is too large. Lower the heap growing factor.
     return judgeGCThreshold(32 * 1024 * 1024, 5.0);
 }
 
+void ThreadState::scheduleV8FollowupGCIfNeeded()
+{
+    ASSERT(checkThread());
+    if (isGCForbidden())
+        return;
+
+    if (isSweepingInProgress())
+        return;
+    ASSERT(!sweepForbidden());
+
+    Heap::reportMemoryUsageForTracing();
+    if (shouldScheduleV8FollowupGC())
+        schedulePreciseGC();
+}
+
 void ThreadState::schedulePageNavigationGCIfNeeded(float estimatedRemovalRatio)
 {
     ASSERT(checkThread());
     if (isGCForbidden())
         return;
 
     // Finish on-going lazy sweeping.
     // TODO(haraken): It might not make sense to force completeSweep() for all
     // page navigations.
     completeSweep();
@@ skipping 902 matching lines @@
         json->beginArray(it->key.ascii().data());
         for (size_t age = 0; age <= maxHeapObjectAge; ++age)
             json->pushInteger(it->value.ages[age]);
         json->endArray();
     }
     TRACE_EVENT_OBJECT_SNAPSHOT_WITH_ID(TRACE_DISABLED_BY_DEFAULT("blink_gc"), statsName, this, json.release());
 }
 #endif
 
 } // namespace blink