Closed
Bug 584298
Opened 14 years ago
Closed 14 years ago
Speed up ReadableToJSVal a bit more
Categories
(Core :: XPConnect, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla2.0b4
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
6.88 KB,
patch
|
jst
:
review+
jst
:
approval2.0+
|
Details | Diff | Splinter Review |
We're penalizing the fast codepaths in favor of the slow ones right now. That seems wrong. Move work out of the guts and into the slow codepaths that actually need it.
Assignee | ||
Comment 1•14 years ago
|
||
Attachment #462699 -
Flags: review?(jst)
Attachment #462699 -
Flags: approval2.0?
Assignee | ||
Updated•14 years ago
|
Summary: Speed up ReadableToJSVal a bit more → [FIX]Speed up ReadableToJSVal a bit more
Assignee | ||
Updated•14 years ago
|
Summary: [FIX]Speed up ReadableToJSVal a bit more → Speed up ReadableToJSVal a bit more
Whiteboard: [need review]
Updated•14 years ago
|
Attachment #462699 -
Flags: approval2.0?
Updated•14 years ago
|
Attachment #462699 -
Flags: review?(jst) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Comment on attachment 462699 [details] [diff] [review] Like so Safe performance win.
Attachment #462699 -
Flags: approval2.0?
Assignee | ||
Updated•14 years ago
|
Whiteboard: [need review] → [need approval]
Updated•14 years ago
|
Attachment #462699 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Updated•14 years ago
|
Whiteboard: [need approval] → [need landing]
Assignee | ||
Comment 3•14 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/4ba110e42f25
Status: NEW → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b4
Assignee | ||
Updated•14 years ago
|
Whiteboard: [need landing]
You need to log in
before you can comment on or make changes to this bug.
Description
•