Closed
Bug 540029
Opened 15 years ago
Closed 15 years ago
Workers: Flush JIT cache when suspended
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bent.mozilla, Assigned: bent.mozilla)
References
Details
Attachments
(1 file)
|
1.02 KB,
patch
|
jst
:
review+
jst
:
superreview+
dveditz
:
approval1.9.2.9-
|
Details | Diff | Splinter Review |
Now that bug 474500 is fixed we should flush the jit cache when we suspend workers. Hopefully that will further reduce the cost of suspended workers.
|
Assignee | |
Comment 1•15 years ago
|
||
Super simple!
Assignee: nobody → bent.mozilla
Status: NEW → ASSIGNED
Attachment #427661 -
Flags: superreview?(jst)
Attachment #427661 -
Flags: review?(jst)
|
||
Updated•15 years ago
|
Attachment #427661 -
Flags: superreview?(jst)
Attachment #427661 -
Flags: superreview+
Attachment #427661 -
Flags: review?(jst)
Attachment #427661 -
Flags: review+
|
|
Assignee | |
Comment 2•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•15 years ago
|
status1.9.2:
final-fixed → ---
|
|
Assignee | |
Comment 3•15 years ago
|
||
Comment on attachment 427661 [details] [diff] [review]
Patch
This saves us JS code cache memory when we suspend workers after a user leaves the page. And it's a super simple patch.
Attachment #427661 -
Flags: approval1.9.2.3?
|
||
Updated•15 years ago
|
Attachment #427661 -
Flags: approval1.9.2.4? → approval1.9.2.8?
|
|
||
Comment 4•15 years ago
|
||
Comment on attachment 427661 [details] [diff] [review]
Patch
The gain from this doesn't seem worth the bother of landing on the branch, unless you're super-eager to see it there.
Attachment #427661 -
Flags: approval1.9.2.9? → approval1.9.2.9-
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•