Refactors parts of gin:

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 27 matching lines @@
       .SetMethod("reset",
                  base::Bind(&base::Timer::Reset, base::Unretained(&timer_)));
 }
 
 Timer::Timer(v8::Isolate* isolate, bool repeating, int delay_ms,
              v8::Handle<v8::Function> function)
     : weak_factory_(this),
       timer_(false, repeating),
       runner_(PerContextData::From(
           isolate->GetCurrentContext())->runner()->GetWeakPtr()) {
-  GetWrapper(runner_->isolate())->SetHiddenValue(GetHiddenPropertyName(isolate),
-                                                 function);
+  GetWrapper(runner_->GetContextHolder()->isolate())->SetHiddenValue(
+      GetHiddenPropertyName(isolate), function);
   timer_.Start(FROM_HERE, base::TimeDelta::FromMilliseconds(delay_ms),
                base::Bind(&Timer::OnTimerFired, weak_factory_.GetWeakPtr()));
 }
 
 Timer::~Timer() {
 }
 
 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::Handle<v8::Function> function = v8::Handle<v8::Function>::Cast(
-      GetWrapper(runner_->isolate())->GetHiddenValue(
-          GetHiddenPropertyName(runner_->isolate())));
-  runner_->Call(function, v8::Undefined(runner_->isolate()), 0, NULL);
+      GetWrapper(isolate)->GetHiddenValue(GetHiddenPropertyName(isolate)));
+  runner_->Call(function, v8::Undefined(isolate), 0, NULL);
 }
 
 
 // TimerModule =================================================================
 
 const char TimerModule::kName[] = "timer";
 WrapperInfo TimerModule::kWrapperInfo = { kEmbedderNativeGin };
 
 // static
 Handle<TimerModule> TimerModule::Create(v8::Isolate* isolate) {
@@ skipping 14 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