Add inspection of function for optimized frames

Add inspection of function for optimized frames

R=svenpanne@chromium.org

BUG=none
TEST=test/mjsunit/debug-evaluate-locals-optimized.js,test/mjsunit/debug-evaluate-locals-optimized-doubles.js

Committed: http://code.google.com/p/v8/source/detail?r=8556

[Sven Panne, 2011-07-06 12:24:19]

LGTM apart from the minor typing issue, which is probably more an aesthetical
remark.

http://codereview.chromium.org/7227006/diff/1/src/deoptimizer.h
File src/deoptimizer.h (right):

http://codereview.chromium.org/7227006/diff/1/src/deoptimizer.h#newcode669
src/deoptimizer.h:669: Object* GetFunction() {
Why should we make the typing weaker here than it is? Returning JSFunction*
seems to be more appropriate, even when we currently only need an Object*.

[Søren Thygesen Gjesse, 2011-07-06 13:01:43]

http://codereview.chromium.org/7227006/diff/1/src/deoptimizer.h
File src/deoptimizer.h (right):

http://codereview.chromium.org/7227006/diff/1/src/deoptimizer.h#newcode669
src/deoptimizer.h:669: Object* GetFunction() {
On 2011/07/06 12:24:19, Sven wrote:
> Why should we make the typing weaker here than it is? Returning JSFunction*
> seems to be more appropriate, even when we currently only need an Object*.

Good point, changed to JSFunction*.