Add support for allowing an embedder to get the V8 profile timer event logs.

src/api.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 383 matching lines @@
 #endif
 }
 
 
 void V8::SetFatalErrorHandler(FatalErrorCallback that) {
   i::Isolate* isolate = EnterIsolateIfNeeded();
   isolate->set_exception_behavior(that);
 }
 
 
+void V8::SetEventLogger(LogEventCallback that) {
+  i::Isolate* isolate = EnterIsolateIfNeeded();
+  isolate->set_event_logger(that);

[Sven Panne, 2014/03/05 07:47:29]

This line is a *very* strong hint that SetEventLogger is in the wrong class: It
should be a normal member function within Isolate. Furthermore, using
EnterIsolateIfNeeded is a hack and should not be used.

[Yang, 2014/03/05 08:18:31]

I agree. I would also suggest adding it to V8::Isolate. 

[fmeawad, 2014/03/06 00:24:46]

Done.

+}
+
+
 void V8::SetAllowCodeGenerationFromStringsCallback(
     AllowCodeGenerationFromStringsCallback callback) {
   i::Isolate* isolate = EnterIsolateIfNeeded();
   isolate->set_allow_code_gen_callback(callback);
 }
 
 
 void V8::SetFlagsFromString(const char* str, int length) {
   i::FlagList::SetFlagsFromString(str, length);
 }
@@ skipping 6963 matching lines @@
   Isolate* isolate = reinterpret_cast<Isolate*>(info.GetIsolate());
   Address callback_address =
       reinterpret_cast<Address>(reinterpret_cast<intptr_t>(callback));
   VMState<EXTERNAL> state(isolate);
   ExternalCallbackScope call_scope(isolate, callback_address);
   callback(info);
 }
 
 
 } }  // namespace v8::internal