Ignore fired timer if the object wrapper is no longer alive

gin/modules/timer.cc

 // Copyright 2013 The Chromium 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 "gin/modules/timer.h"
 
 #include "base/bind.h"
 #include "gin/object_template_builder.h"
 #include "gin/per_context_data.h"
 
@@ skipping 49 matching lines @@
 void Timer::OnTimerFired() {
   // This can happen in spite of the weak callback because it is possible for
   // a gin::Handle<> to keep this object alive past when the isolate it is part
   // of is destroyed.
   if (!runner_.get()) {
     return;
   }
 
   Runner::Scope scope(runner_.get());
   v8::Isolate* isolate = runner_->GetContextHolder()->isolate();
+
+  v8::Local<v8::Object> wrapper;
+  if (!GetWrapper(isolate).ToLocal(&wrapper)) {
+    return;
+  }
+
   v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(
-      GetWrapper(isolate)
-          .ToLocalChecked()
+      wrapper
           ->GetPrivate(runner_->GetContextHolder()->context(),
                        GetHiddenPropertyName(isolate))
           .ToLocalChecked());
   runner_->Call(function, v8::Undefined(isolate), 0, NULL);
 }
 
 
 // TimerModule =================================================================
 
 const char TimerModule::kName[] = "timer";
@@ skipping 18 matching lines @@
 ObjectTemplateBuilder TimerModule::GetObjectTemplateBuilder(
     v8::Isolate* isolate) {
   return Wrappable<TimerModule>::GetObjectTemplateBuilder(isolate)
       .SetMethod("createOneShot",
                  base::Bind(&Timer::Create, Timer::TYPE_ONE_SHOT))
       .SetMethod("createRepeating",
                  base::Bind(&Timer::Create, Timer::TYPE_REPEATING));
 }
 
 }  // namespace gin