Last Comment Bug 873356 - Should do shrinking GCs more often for workers
: Should do shrinking GCs more often for workers
Status: RESOLVED FIXED
[MemShrink:P1]
:
Product: Core
Classification: Components
Component: DOM: Workers (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla24
Assigned To: Ben Turner (not reading bugmail, use the needinfo flag!)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-16 21:37 PDT by Ben Turner (not reading bugmail, use the needinfo flag!)
Modified: 2013-06-07 08:34 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch, v1 (68.55 KB, patch)
2013-05-16 21:37 PDT, Ben Turner (not reading bugmail, use the needinfo flag!)
khuey: review+
Details | Diff | Review

Description Ben Turner (not reading bugmail, use the needinfo flag!) 2013-05-16 21:37:51 PDT
Created attachment 750872 [details] [diff] [review]
Patch, v1

This is the mozilla-central version of bug 829482. This re-syncs all the main thread prefs for use with workers and allows overrides for workers if needed. Also updates the priority of workers depending on whether they're chrome or content.
Comment 1 Justin Lebar (not reading bugmail) 2013-05-29 12:07:38 PDT
Stupid question, but do we do a shrinking-gc in workers upon receiving a memory-pressure event?
Comment 2 Ben Turner (not reading bugmail, use the needinfo flag!) 2013-05-29 12:09:42 PDT
Yep, http://mxr.mozilla.org/mozilla-central/source/dom/workers/RuntimeService.cpp#1434
Comment 3 Ben Turner (not reading bugmail, use the needinfo flag!) 2013-06-06 13:49:38 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/f7c514f26232
Comment 4 Ryan VanderMeulen [:RyanVM] 2013-06-07 08:34:03 PDT
https://hg.mozilla.org/mozilla-central/rev/f7c514f26232

Note You need to log in before you can comment on or make changes to this bug.