Revision: 2575
Author: [email protected]
Date: Wed Jul 29 04:55:26 2009
Log: Use JSFunction::IsBuiltin() for checking if a function is a builtin one
Review URL: http://codereview.chromium.org/160323
http://code.google.com/p/v8/source/detail?r=2575

Modified:
  /branches/bleeding_edge/src/execution.cc

=======================================
--- /branches/bleeding_edge/src/execution.cc    Thu Jul 23 23:14:23 2009
+++ /branches/bleeding_edge/src/execution.cc    Wed Jul 29 04:55:26 2009
@@ -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
-~----------~----~----~----~------~----~------~--~---

Reply via email to