Auto create ApiLocalScope before calling native functions, this ensures that

runtime/vm/benchmark_test.cc

 // 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.
 
 #include "vm/benchmark_test.h"
 
 #include "bin/builtin.h"
 #include "bin/file.h"
 
 #include "platform/assert.h"
@@ skipping 88 matching lines @@
   Dart_Handle result = Dart_SetNativeInstanceField(recv, 0, 7);
   EXPECT_VALID(result);
 
   Dart_ExitScope();
 }
 
 
 // The specific api functions called here are a bit arbitrary.  We are
 // trying to get a sense of the overhead for using the dart api.
 static void UseDartApi(Dart_NativeArguments args) {
-  Dart_EnterScope();
   int count = Dart_GetNativeArgumentCount(args);
   EXPECT_EQ(3, count);
 
   // Get native field from receiver.
   intptr_t receiver_value;
   Dart_Handle result = Dart_GetNativeReceiver(args, &receiver_value);
   EXPECT_VALID(result);
   EXPECT_EQ(7, receiver_value);
 
   // Get param1.
   Dart_Handle param1 = Dart_GetNativeArgument(args, 1);
   EXPECT_VALID(param1);
   EXPECT(Dart_IsInteger(param1));
   bool fits = false;
   result = Dart_IntegerFitsIntoInt64(param1, &fits);
   EXPECT_VALID(result);
   EXPECT(fits);
   int64_t value1;
   result = Dart_IntegerToInt64(param1, &value1);
   EXPECT_VALID(result);
   EXPECT_LE(0, value1);
   EXPECT_LE(value1, 1000000);
 
   // Return param + receiver.field.
   Dart_SetReturnValue(args, Dart_NewInteger(value1 * receiver_value));
-  Dart_ExitScope();
 }
 
 
 static Dart_NativeFunction bm_uda_lookup(Dart_Handle name, int argument_count) {
   const char* cstr = NULL;
   Dart_Handle result = Dart_StringToCString(name, &cstr);
   EXPECT_VALID(result);
   if (strcmp(cstr, "init") == 0) {
     return InitNativeFields;
   } else {
@@ skipping 153 matching lines @@
   free(dart_root);
   free(script);
 }
 
 
 //
 // Measure frame lookup during stack traversal.
 //
 static void StackFrame_accessFrame(Dart_NativeArguments args) {
   const int kNumIterations = 100;
-  Dart_EnterScope();
   Code& code = Code::Handle();
   Timer timer(true, "LookupDartCode benchmark");
   timer.Start();
   for (int i = 0; i < kNumIterations; i++) {
     StackFrameIterator frames(StackFrameIterator::kDontValidateFrames);
     StackFrame* frame = frames.NextFrame();
     while (frame != NULL) {
       if (frame->IsStubFrame()) {
         code = frame->LookupDartCode();
         EXPECT(code.function() == Function::null());
       } else if (frame->IsDartFrame()) {
         code = frame->LookupDartCode();
         EXPECT(code.function() != Function::null());
       }
       frame = frames.NextFrame();
     }
   }
   timer.Stop();
   int64_t elapsed_time = timer.TotalElapsedTime();
   Dart_SetReturnValue(args, Dart_NewInteger(elapsed_time));
-  Dart_ExitScope();
 }
 
 
 static Dart_NativeFunction StackFrameNativeResolver(Dart_Handle name,
                                                     int arg_count) {
   return &StackFrame_accessFrame;
 }
 
 
 // Unit test case to verify stack frame iteration.
@@ skipping 123 matching lines @@
 
   Timer timer(true, "currentMirrorSystem() benchmark");
   timer.Start();
   Dart_Invoke(lib, NewString("benchmark"), 0, NULL);
   timer.Stop();
   int64_t elapsed_time = timer.TotalElapsedTime();
   benchmark->set_score(elapsed_time);
 }
 
 }  // namespace dart