Reaction calculation 4190

resolve the merge conflict...
Ref #4190 
@cpritam, does it make sense to leave it to be decided when you are moving jacobian_mult out, to make sure all these tmp variables are kept as a whole?