Closed Bug 488080 Opened 11 years ago Closed 11 years ago
Workers: Don't compile script if the worker is already canceled
We can avoid a little work here. Small perf improvement on bug 487465.
Comment on attachment 372368 [details] [diff] [review] Patch r+sr=jst As a side note, should the methods in this class be un-inlined? A bunch are virtual, some are fairly substantial etc.
Actually, that patch breaks the test_closeOnGC test. This is better.
Pushed changeset f2a2bb94bf9a to mozilla-central.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
11 years ago
Comment on attachment 394122 [details] [diff] [review] Patch, v2 approval1.9.2 requests aren't currently being monitored, since we're nearing RC freeze and there are too many outstanding requests, so I'm clearing this request. Feel free to re-request approval if you are confident that it's worth drivers' time to consider whether this non-blocker needs to land for 1.9.2 at this stage.
Comment on attachment 394122 [details] [diff] [review] Patch, v2 I'd like to put this on the branch. It fixes an easy to reproduce slowdown that can make shutdown take way longer than necessary, and it's been tested on trunk for quite a while now.
Attachment #394122 - Flags: approval126.96.36.199?
Comment on attachment 394122 [details] [diff] [review] Patch, v2 a1922=beltzner
Attachment #394122 - Flags: approval188.8.131.52? → approval184.108.40.206+
You need to log in before you can comment on or make changes to this bug.