Removed V8WorkerOrWorkletEventListener::HandleEvent()

Removed V8WorkerOrWorkletEventListener::HandleEvent() in order to
remove blink::V8WorkerOrWorkletEventListener.
The difference between V8AbstractEventListener::HandleEvent() and
V8WorkerOrWorkletEventListener::HandleEvent() was checking if script
controller is available or not. This should be done in not only
V8WorkerOrWorkletEventListener but also other event listeners, so this
CL added this operation by checking ScriptState::ContextIsValid().

Bug: 869778
Change-Id: I3168f9990d2c6d076061b4d7d8f9ab85619f71d5
Reviewed-on: https://chromium-review.googlesource.com/1160069Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Commit-Queue: Yuki Yamada <yukiy@google.com>
Cr-Commit-Position: refs/heads/master@{#580808}

third_party/blink/renderer/bindings/core/v8/v8_abstract_event_listener.cc

@@ -114,6 +114,9 @@ void V8AbstractEventListener::handleEvent(ExecutionContext* execution_context,
 
 void V8AbstractEventListener::HandleEvent(ScriptState* script_state,
                                           Event* event) {
+  if (!script_state->ContextIsValid())
+    return;
+
   ScriptState::Scope scope(script_state);
 
   // Get the V8 wrapper for the event object.

third_party/blink/renderer/bindings/core/v8/v8_worker_or_worklet_event_listener.cc

@@ -49,26 +49,6 @@ V8WorkerOrWorkletEventListener::V8WorkerOrWorkletEventListener(
     ScriptState* script_state)
     : V8EventListener(is_inline, script_state) {}
 
-void V8WorkerOrWorkletEventListener::HandleEvent(ScriptState* script_state,
-                                                 Event* event) {
-  v8::Local<v8::Context> context = script_state->GetContext();
-  WorkerOrWorkletScriptController* script_controller =
-      ToWorkerOrWorkletGlobalScope(ToExecutionContext(context))
-          ->ScriptController();
-  if (!script_controller)
-    return;
-
-  ScriptState::Scope scope(script_state);
-
-  // Get the V8 wrapper for the event object.
-  v8::Local<v8::Value> js_event = ToV8(event, context->Global(), GetIsolate());
-  if (js_event.IsEmpty())
-    return;
-
-  InvokeEventHandler(script_state, event,
-                     v8::Local<v8::Value>::New(GetIsolate(), js_event));
-}
-
 v8::Local<v8::Value> V8WorkerOrWorkletEventListener::CallListenerFunction(
     ScriptState* script_state,
     v8::Local<v8::Value> js_event,

third_party/blink/renderer/bindings/core/v8/v8_worker_or_worklet_event_listener.h

@@ -52,8 +52,6 @@ class V8WorkerOrWorkletEventListener final : public V8EventListener {
     return event_listener;
   }
 
-  void HandleEvent(ScriptState*, Event*) override;
-
  protected:
   V8WorkerOrWorkletEventListener(bool is_inline, ScriptState*);