[Interpreter] Add support for FunctionEntryHook.

test/cctest/test-api.cc

Index: test/cctest/test-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index 6bea3eaefee603ed7bc02a56be251082ba6ad27a..fa22557e28d196c4d6ccb029ba4152614c36396a 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -14403,7 +14403,6 @@ int SetFunctionEntryHookTest::CountInvocations(
   return invocations;
 }
 
-
 void SetFunctionEntryHookTest::RunLoopInNewEnv(v8::Isolate* isolate) {
   v8::HandleScope outer(isolate);
   v8::Local<Context> env = Context::New(isolate);
@@ -14461,10 +14460,18 @@ void SetFunctionEntryHookTest::RunTest() {
 
     RunLoopInNewEnv(isolate);
 
-    // Check the exepected invocation counts.
-    CHECK_EQ(2, CountInvocations(NULL, "bar"));
-    CHECK_EQ(200, CountInvocations("bar", "foo"));
-    CHECK_EQ(200, CountInvocations(NULL, "foo"));
+    // Check the expected invocation counts.
+    if (!i::FLAG_ignition) {
+      CHECK_EQ(2, CountInvocations(NULL, "bar"));
+      CHECK_EQ(200, CountInvocations("bar", "foo"));
+      CHECK_EQ(200, CountInvocations(NULL, "foo"));
+    } else {
+      // For ignition we don't see the actual functions being called, instead
+      // we see the IterpreterEntryTrampoline.
+      CHECK_NE(0, CountInvocations(NULL, "InterpreterEntryTrampoline"));

[Michael Starzinger, 2016/04/15 13:42:56]

Can this be made stricter by saying checking that we called the trampoline at
least 100 times? I was thinking about ...

CHECK_LE(100, CountInvocations(NULL, "InterpreterEntryTrampoline"));

[rmcilroy, 2016/04/18 09:25:43]

Went with the minimum of 102 (100 calls of foo and 2 of bar) to be even stricter
:).

+      // We should also see the calls to the optimized function foo.
+      CHECK_EQ(100, CountInvocations(NULL, "foo"));
+    }
 
     // Verify that we have an entry hook on some specific stubs.
     CHECK_NE(0, CountInvocations(NULL, "CEntryStub"));