[inspector] introduced debug::SetCompileEventListener

src/debug/debug.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/debug/debug.h"
 
 #include <memory>
 
 #include "src/api.h"
 #include "src/arguments.h"
@@ skipping 1848 matching lines @@
   return async_id->value();
 }
 
 void Debug::SetAsyncTaskListener(debug::AsyncTaskListener listener,
                                  void* data) {
   async_task_listener_ = listener;
   async_task_listener_data_ = data;
   UpdateState();
 }
 
+void Debug::SetCompileEventListener(debug::CompileEventListener listener,
+                                    void* data) {
+  compile_event_listener_ = listener;
+  compile_event_listener_data_ = data;
+  UpdateState();
+}
+
 void Debug::OnAsyncTaskEvent(debug::PromiseDebugActionType type, int id) {
   if (in_debug_scope() || ignore_events()) return;
 
   if (async_task_listener_) {
     async_task_listener_(type, id, async_task_listener_data_);
     // There are three types of event listeners: C++ message_handler,
     // JavaScript event listener and C++ event listener.
     // Currently inspector still uses C++ event listener and installs
     // more specific event listeners for part of events. Calling of
     // C++ event listener is redundant when more specific event listener
@@ skipping 81 matching lines @@
   in_debug_event_listener_ = previous;
 }
 
 
 void Debug::ProcessCompileEvent(v8::DebugEvent event, Handle<Script> script) {
   if (ignore_events()) return;
   if (script->type() != i::Script::TYPE_NORMAL &&
       script->type() != i::Script::TYPE_WASM) {
     return;
   }
+  if (compile_event_listener_) {
+    compile_event_listener_(ToApiHandle<debug::Script>(script),
+                            event != v8::AfterCompile,
+                            compile_event_listener_data_);
+    bool non_inspector_listener_exists =
+        message_handler_ != nullptr ||

[jgruber, 2017/01/16 19:26:32]

Nit: Maybe it'd make sense to extract this to a method at some point.

[kozy, 2017/01/16 22:22:39]

Done.

+        (event_listener_.is_null() && !event_listener_->IsForeign());
+    if (!non_inspector_listener_exists) return;
+  }
+
   SuppressDebug while_processing(this);
 
   bool in_nested_debug_scope = in_debug_scope();
   HandleScope scope(isolate_);
   DebugScope debug_scope(this);
   if (debug_scope.failed()) return;
 
   // Create the compile state object.
   Handle<Object> event_data;
   // Bail out and don't call debugger if exception.
@@ skipping 180 matching lines @@
   UpdateState();
   if (handler == NULL && in_debug_scope()) {
     // Send an empty command to the debugger if in a break to make JavaScript
     // run again if the debugger is closed.
     EnqueueCommandMessage(Vector<const uint16_t>::empty());
   }
 }
 
 void Debug::UpdateState() {
   bool is_active = message_handler_ != nullptr || !event_listener_.is_null() ||
-                   async_task_listener_ != nullptr;
+                   async_task_listener_ != nullptr ||
+                   compile_event_listener_ != nullptr;
   if (is_active || in_debug_scope()) {
     // Note that the debug context could have already been loaded to
     // bootstrap test cases.
     isolate_->compilation_cache()->Disable();
     is_active = Load();
   } else if (is_loaded()) {
     isolate_->compilation_cache()->Enable();
     Unload();
   }
   is_active_ = is_active;
@@ skipping 467 matching lines @@
   logger_->DebugEvent("Put", message.text());
 }
 
 void LockingCommandMessageQueue::Clear() {
   base::LockGuard<base::Mutex> lock_guard(&mutex_);
   queue_.Clear();
 }
 
 }  // namespace internal
 }  // namespace v8