Created attachment 525564 [details] [diff] [review]
This patch factors out some prep work for bug 644074.
0. Inline ComputeGlobalThis into its only caller
1. Split CallArgs into CallReceiver and CallArgs to capture/remove the uses of 'vp' where we don't have/care about argv.
2. Assert call.callee() isn't used after call.rval()
3. Change JSStackFrame::computeThis into a free function since isn't a simple accessor/mutator of a stack frame; its an effectful interpreter operation akin to ComputeImplicitThis/GetPrimitiveThis/BoxNonStrictThis. Put it next to these functions.
4. Reunite GetPrimitiveThis and details::PrimitiveBehavior
Created attachment 526092 [details] [diff] [review]
this time qref'd