bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Consider to not run 40ms GC slices in parent process when user is interacting with the browser

RESOLVED FIXED in Firefox 54

Status

()

Core
DOM
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

50 Branch
mozilla54
Points:
---

Firefox Tracking Flags

(firefox54 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

If input events go to a child process, nothing in parent is possibly painted and refresh driver doesn't tick there, meaning that it uses longer gc slices. Yet that may affect when the input events are sent to the child, or if child does some sync IPC message to parent, it may be blocked because of GC.
Created attachment 8836118 [details] [diff] [review]
wip

Is this a silly idea?
Assignee: nobody → bugs
Attachment #8836118 - Flags: review?(jcoppeard)
Comment on attachment 8836118 [details] [diff] [review]
wip

Review of attachment 8836118 [details] [diff] [review]:
-----------------------------------------------------------------

This sounds like a good idea to me.

I don't know if we've got any way of measuring responsiveness in this situation.  But if this is the way that things worked pre-e10s then I think we should do that.
Attachment #8836118 - Flags: review?(jcoppeard) → review+

Comment 3

a year ago
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/51ba586bd63a
make parent process to use 10ms gc slices when user is active, r=jonco

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/51ba586bd63a
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Comment hidden (mozreview-request)
(Assignee)

Comment 6

a year ago
mozreview-review
Comment on attachment 8836470 [details]
Bug 1338518 - Followup: Fix incorrect `if` block in SetMemoryGCSliceTimePrefChangedCallback().

https://reviewboard.mozilla.org/r/111888/#review113148

oops, thanks.
Attachment #8836470 - Flags: review?(bugs) → review+

Comment 7

a year ago
Comment on attachment 8836470 [details]
Bug 1338518 - Followup: Fix incorrect `if` block in SetMemoryGCSliceTimePrefChangedCallback().

Thanks.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8fba5c26a855cc121f60a6545e2b83ca38e6c358
Attachment #8836470 - Flags: checkin?
Attachment #8836470 - Flags: checkin?

Updated

a year ago
Attachment #8836470 - Attachment is obsolete: true
Comment on attachment 8836470 [details]
Bug 1338518 - Followup: Fix incorrect `if` block in SetMemoryGCSliceTimePrefChangedCallback().

(This landed, so, it is not obsolete)
Attachment #8836470 - Attachment is obsolete: false
You need to log in before you can comment on or make changes to this bug.