Use smaller heap growing factor in idle notification to start incremental marking when there is idl…

src/heap/gc-idle-time-handler.h

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_HEAP_GC_IDLE_TIME_HANDLER_H_
 #define V8_HEAP_GC_IDLE_TIME_HANDLER_H_
 
 #include "src/globals.h"
 
 namespace v8 {
@@ skipping 104 matching lines @@
   // idle round.
   static const int kMaxMarkCompactsInIdleRound;
 
   // Number of scavenges that will trigger start of new idle round.
   static const int kIdleScavengeThreshold;
 
   // This is the maximum scheduled idle time. Note that it can be more than
   // 16 ms when there is currently no rendering going on.
   static const size_t kMaxScheduledIdleTime = 50;
 
+  // The maximum idle time when frames are rendered is 16ms.
+  static const size_t kMaxFrameRenderingIdleTime = 16;

[Erik Corry Chromium.org, 2015/04/16 14:16:44]

It's 16 and two thirds.  I think you should go with 17 because rounding errors.

[Hannes Payer (out of office), 2015/04/16 14:46:45]

Done.

[Erik Corry, 2015/04/16 15:35:46]

It's actually 16⅔.  Should we use 17 here?

+
   // We conservatively assume that in the next kTimeUntilNextIdleEvent ms
   // no idle notification happens.
   static const size_t kTimeUntilNextIdleEvent = 100;
 
   // If we haven't recorded any scavenger events yet, we use a conservative
   // lower bound for the scavenger speed.
   static const size_t kInitialConservativeScavengeSpeed = 100 * KB;
 
   // If contexts are disposed at a higher rate a full gc is triggered.
   static const double kHighContextDisposalRate;
@@ skipping 93 matching lines @@
   int scavenges_since_last_idle_round_;
   int idle_times_which_made_no_progress_since_last_idle_round_;
 
   DISALLOW_COPY_AND_ASSIGN(GCIdleTimeHandler);
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_HEAP_GC_IDLE_TIME_HANDLER_H_