No need in require_v8_scope flag any more.

tools/dom/scripts/systemnative.py

 #!/usr/bin/python
 # Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 """This module provides shared functionality for the systems to generate
 native binding from the IDL database."""
 
 import emitter
 import os
@@ skipping 525 matching lines @@
       callback_name,
       needs_receiver,
       function_expression,
       node,
       arguments,
       return_type,
       raises_dom_exception):
     ext_attrs = node.ext_attrs
 
     cpp_arguments = []
-    requires_v8_scope = \
-        any((self._TypeInfo(argument.type.id).requires_v8_scope() for argument in arguments)) or\
-        self._interface.id.startswith('IDB')
     runtime_check = None
     raises_exceptions = raises_dom_exception or arguments
 
     # TODO(antonm): unify with ScriptState below.
     requires_stack_info = ext_attrs.get('CallWith') == 'ScriptArguments|ScriptState'
     if requires_stack_info:
       raises_exceptions = True
-      requires_v8_scope = True
       cpp_arguments = ['&state', 'scriptArguments.release()']
       # WebKit uses scriptArguments to reconstruct last argument, so
       # it's not needed and should be just removed.
       arguments = arguments[:-1]
 
     # TODO(antonm): unify with ScriptState below.
     requires_script_arguments = ext_attrs.get('CallWith') == 'ScriptArguments'
     if requires_script_arguments:
       raises_exceptions = True
-      requires_v8_scope = True
       cpp_arguments = ['scriptArguments.release()']
       # WebKit uses scriptArguments to reconstruct last argument, so
       # it's not needed and should be just removed.
       arguments = arguments[:-1]
 
     requires_script_execution_context = ext_attrs.get('CallWith') == 'ScriptExecutionContext'
     if requires_script_execution_context:
       raises_exceptions = True
       cpp_arguments = ['context']
 
@@ skipping 57 matching lines @@
           'fail:\n'
           '    Dart_ThrowException(exception);\n'
           '    ASSERT_NOT_REACHED();\n')
 
     body_emitter = body_emitter.Emit(
         '    {\n'
         '$!BODY'
         '        return;\n'
         '    }\n')
 
-    if requires_v8_scope:
-      body_emitter.Emit(
-          '        V8Scope v8scope;\n\n')
-
     if runtime_check:
       body_emitter.Emit(
           '$RUNTIME_CHECK\n',
           RUNTIME_CHECK=runtime_check)
 
     if requires_script_execution_context:
       body_emitter.Emit(
           '        ScriptExecutionContext* context = DartUtilities::scriptExecutionContext();\n'
           '        if (!context) {\n'
           '            exception = Dart_NewStringFromCString("Failed to retrieve a context");\n'
@@ skipping 249 matching lines @@
         LIBRARY_NAME=library_name)
 
       headers = self._library_headers[library_name]
       for header_file in headers:
         path = os.path.relpath(header_file, output_dir)
         includes_emitter.Emit('#include "$PATH"\n', PATH=path)
         body_emitter.Emit(
             '    if (Dart_NativeFunction func = $CLASS_NAME::resolver(name, argumentCount))\n'
             '        return func;\n',
             CLASS_NAME=os.path.splitext(os.path.basename(path))[0])