[Refactoring] Clean up binding template files for callback interfaces

third_party/WebKit/Source/bindings/templates/callback_interface.cpp.tmpl

 {% filter format_blink_cpp_source_code %}
 
 {% include 'copyright_block.txt' %}
 #include "{{v8_class}}.h"
 
 {% for filename in cpp_includes %}
 #include "{{filename}}"
 {% endfor %}
 
 namespace blink {
@@ skipping 12 matching lines @@
 {% for method in methods if not method.is_custom %}
 {{method.cpp_type}} {{v8_class}}::{{method.name}}({{method.argument_declarations | join(', ')}}) {
   {% set return_default = 'return true'
          if method.idl_type == 'boolean' else 'return' %}{# void #}
   ExecutionContext* executionContext = m_scriptState->getExecutionContext();
   if (!executionContext || executionContext->isContextSuspended() ||
       executionContext->isContextDestroyed())
     {{return_default}};
   if (!m_scriptState->contextIsValid())
     {{return_default}};
+
   ScriptState::Scope scope(m_scriptState.get());
   {% if method.call_with_this_handle %}
   v8::Local<v8::Value> thisHandle = thisValue.v8Value();
   {% endif %}
+
   {% for argument in method.arguments %}
   v8::Local<v8::Value> {{argument.handle}} = {{argument.cpp_value_to_v8_value}};
   {% endfor %}
   {% if method.arguments %}
   v8::Local<v8::Value> argv[] = { {{method.arguments | join(', ', 'handle')}} };
   {% else %}
   {# Empty array initializers are illegal, and don\'t compile in MSVC. #}
   v8::Local<v8::Value> *argv = 0;
   {% endif %}
 
-  {% set this_handle_parameter = 'thisHandle, ' if method.call_with_this_handle else 'v8::Undefined(m_scriptState->isolate()), ' %}
+  v8::Isolate* isolate = m_scriptState->isolate();
+  {% set this_handle_parameter = 'thisHandle'
+         if method.call_with_this_handle else 'v8::Undefined(isolate)' %}
   {% if method.idl_type == 'boolean' %}
-  v8::TryCatch exceptionCatcher(m_scriptState->isolate());
+  v8::TryCatch exceptionCatcher(isolate);
   exceptionCatcher.SetVerbose(true);
-  V8ScriptRunner::callFunction(m_callback.newLocal(m_scriptState->isolate()), m_scriptState->getExecutionContext(), {{this_handle_parameter}}{{method.arguments | length}}, argv, m_scriptState->isolate());
+  V8ScriptRunner::callFunction(m_callback.newLocal(isolate),
+                               executionContext,
+                               {{this_handle_parameter}},
+                               {{method.arguments | length}},
+                               argv,
+                               isolate);
   return !exceptionCatcher.HasCaught();
   {% else %}{# void #}
-  V8ScriptRunner::callFunction(m_callback.newLocal(m_scriptState->isolate()), m_scriptState->getExecutionContext(), {{this_handle_parameter}}{{method.arguments | length}}, argv, m_scriptState->isolate());
-    {% endif %}
+  V8ScriptRunner::callFunction(m_callback.newLocal(isolate),
+                               m_scriptState->getExecutionContext(),
+                               {{this_handle_parameter}},
+                               {{method.arguments | length}},
+                               argv,
+                               isolate);
+  {% endif %}
 }
 
 {% endfor %}
 }  // namespace blink
 
 {% endfilter %}{# format_blink_cpp_source_code #}