Confirmed users
367
edits
JSStackFrame Evisceration: Difference between revisions
Jump to navigation
Jump to search
→Removing JSStackFrame members / things on the call path
(Created page with '== Removing JSStackFrame members / things on the call path<br> == This would leave sizeof(JSStackFrame) == 6 words.<br> === Directly remove<br> === *[https://bugzilla.mozilla.…') |
|||
| Line 1: | Line 1: | ||
== Removing JSStackFrame members / things on the call path | == Removing JSStackFrame members / things on the call path == | ||
This would leave sizeof(JSStackFrame) == 6 words. | This would leave sizeof(JSStackFrame) == 6 words. | ||
Sorted in estimated order of benefit / difficulty: | |||
=== | {| width="95%" cellspacing="4" cellpadding="1" border="0" | ||
|- | |||
| '''Task''' | |||
| '''Size (wks)''' | |||
| '''Candidate Assignee''' | |||
|- | |||
| [https://bugzilla.mozilla.org/show_bug.cgi?id=539144 argc/argv] | |||
| 1 | |||
| lw | |||
|- | |||
| [https://bugzilla.mozilla.org/show_bug.cgi?id=557375 thisv] (depends on argv) | |||
| .5 | |||
| lw | |||
|- | |||
| fun, script (depend on thisv) | |||
| .5 | |||
| lw | |||
|- | |||
| ncode (merge with savedPC) | |||
| .1 | |||
| dvander | |||
|- | |||
| displaySave - remove display optimization | |||
| 1 | |||
| cdleary | |||
|- | |||
| hookData (just use js::Invoke and use a local variable) | |||
| .5 | |||
| | |||
|- | |||
| [https://bugzilla.mozilla.org/show_bug.cgi?id=540675 callerVersion] | |||
| .5 | |||
| - | |||
| [https://bugzilla.mozilla.org/show_bug.cgi?id=535912 blockChain] | |||
| 1 | |||
| | |||
|- | |||
| [https://bugzilla.mozilla.org/show_bug.cgi?id=550639 rval] | |||
| 1 | |||
| | |||
|- | |||
| annotation | |||
| ? (0, once we can drop the callers of JS_{Get,Set}FrameAnnotation in nsScriptSecurityManager.cpp) | |||
| | |||
|- | |||
| imacpc (use some side stack, or push a jsval (like JSOP_GOSUB) | |||
| 1 | |||
| | |||
|} | |||
== Remove from call path == | == Remove from call path == | ||