Reviewers: Kasper Lund, Description: Use JSFunction::IsBuiltin() for checking if a function is a builtin one
Please review this at http://codereview.chromium.org/160323 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/execution.cc Index: src/execution.cc =================================================================== --- src/execution.cc (revision 2574) +++ src/execution.cc (working copy) @@ -592,14 +592,14 @@ ASSERT(!it.done()); Object* fun = it.frame()->function(); if (fun && fun->IsJSFunction()) { - GlobalObject* global = JSFunction::cast(fun)->context()->global(); // Don't stop in builtin functions. - if (global == Top::context()->builtins()) { - return Heap::undefined_value(); + if (JSFunction::cast(fun)->IsBuiltin()) { + return Heap::undefined_value(); } + GlobalObject* global = JSFunction::cast(fun)->context()->global(); // Don't stop in debugger functions. if (Debug::IsDebugGlobal(global)) { - return Heap::undefined_value(); + return Heap::undefined_value(); } } } --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
