Closed Bug 540029 Opened 15 years ago Closed 14 years ago

Workers: Flush JIT cache when suspended

Categories

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

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bent.mozilla, Assigned: bent.mozilla)

References

Details

Attachments

(1 file)

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.
Attached patch PatchSplinter Review
Super simple!
Assignee: nobody → bent.mozilla
Status: NEW → ASSIGNED
Attachment #427661 - Flags: superreview?(jst)
Attachment #427661 - Flags: review?(jst)
Attachment #427661 - Flags: superreview?(jst)
Attachment #427661 - Flags: superreview+
Attachment #427661 - Flags: review?(jst)
Attachment #427661 - Flags: review+
http://hg.mozilla.org/mozilla-central/rev/c3172dd1a686
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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?
Attachment #427661 - Flags: approval1.9.2.4? → approval1.9.2.8?
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-
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: