502374 - Don't call cycle collector so often

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

This adds the 10s limit to page loads and user inactivity.
At least tryserver didn't show tp regression.

Comment 1

[Peter Van der Beken [:peterv]]

Comment on attachment 386823 [details] [diff] [review]
patch

>diff --git a/dom/base/nsJSEnvironment.cpp b/dom/base/nsJSEnvironment.cpp

>+nsJSContext::IntervalCC()
>+{
>+  if ((PR_Now() - sPreviousCCTime) >=
>+      PRTime(NS_MIN_CC_INTERVAL * PR_USEC_PER_MSEC)) {
>+    nsJSContext::CC();
>+    return PR_TRUE;
>+  }
>+#ifdef DEBUG_smaug
>+  else {
>+    printf("Running CC was delayed because of NS_MIN_CC_INTERVAL.\n");
>+  }

Don't really need the else (after return) here.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Attachment: +comments

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

http://hg.mozilla.org/mozilla-central/rev/96b06aeae4d7