Rename Api::NewLocalHandle -> Api::NewHandle.

runtime/vm/dart_api_impl_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 "include/dart_api.h"
 #include "platform/assert.h"
 #include "platform/utils.h"
 #include "vm/class_finalizer.h"
 #include "vm/dart_api_impl.h"
 #include "vm/dart_api_state.h"
@@ skipping 147 matching lines @@
   EXPECT(!same);
 
   // Different objects.
   EXPECT_VALID(Dart_IsSame(five, seven, &same));
   EXPECT(!same);
 
   // Non-instance objects.
   {
     Isolate* isolate = Isolate::Current();
     DARTSCOPE_NOCHECKS(isolate);
-    const Object& cls1 = Object::Handle(Object::null_class());
-    const Object& cls2 = Object::Handle(Object::class_class());
-    Dart_Handle class1 = Api::NewLocalHandle(isolate, cls1);
-    Dart_Handle class2 = Api::NewLocalHandle(isolate, cls2);
+    Dart_Handle class1 = Api::NewHandle(isolate, Object::null_class());
+    Dart_Handle class2 = Api::NewHandle(isolate, Object::class_class());
 
     EXPECT_VALID(Dart_IsSame(class1, class1, &same));
     EXPECT(same);
 
     EXPECT_VALID(Dart_IsSame(class1, class2, &same));
     EXPECT(!same);
   }
 }
 
 
@@ skipping 940 matching lines @@
   Isolate* isolate = Isolate::Current();
   EXPECT(isolate != NULL);
   ApiState* state = isolate->api_state();
   EXPECT(state != NULL);
   ApiLocalScope* scope = state->top_scope();
   Dart_EnterScope();
   {
     EXPECT(state->top_scope() != NULL);
     DARTSCOPE_NOCHECKS(isolate);
     const String& str1 = String::Handle(String::New("Test String"));
-    Dart_Handle ref = Api::NewLocalHandle(isolate, str1);
+    Dart_Handle ref = Api::NewHandle(isolate, str1.raw());
     String& str2 = String::Handle();
     str2 ^= Api::UnwrapHandle(ref);
     EXPECT(str1.Equals(str2));
   }
   Dart_ExitScope();
   EXPECT(scope == state->top_scope());
 }
 
 
 // Unit test for creating and deleting persistent handles.
 UNIT_TEST_CASE(PersistentHandles) {
   const char* kTestString1 = "Test String1";
   const char* kTestString2 = "Test String2";
   TestCase::CreateTestIsolate();
   Isolate* isolate = Isolate::Current();
   EXPECT(isolate != NULL);
   ApiState* state = isolate->api_state();
   EXPECT(state != NULL);
   ApiLocalScope* scope = state->top_scope();
   Dart_Handle handles[2000];
   Dart_EnterScope();
   {
     DARTSCOPE_NOCHECKS(isolate);
-    const String& str1 = String::Handle(String::New(kTestString1));
-    Dart_Handle ref1 = Api::NewLocalHandle(isolate, str1);
+    Dart_Handle ref1 = Api::NewHandle(isolate, String::New(kTestString1));
     for (int i = 0; i < 1000; i++) {
       handles[i] = Dart_NewPersistentHandle(ref1);
     }
     Dart_EnterScope();
-    const String& str2 = String::Handle(String::New(kTestString2));
-    Dart_Handle ref2 = Api::NewLocalHandle(isolate, str2);
+    Dart_Handle ref2 = Api::NewHandle(isolate, String::New(kTestString2));
     for (int i = 1000; i < 2000; i++) {
       handles[i] = Dart_NewPersistentHandle(ref2);
     }
     for (int i = 500; i < 1500; i++) {
       Dart_DeletePersistentHandle(handles[i]);
     }
     for (int i = 500; i < 1000; i++) {
       handles[i] = Dart_NewPersistentHandle(ref2);
     }
     for (int i = 1000; i < 1500; i++) {
@@ skipping 76 matching lines @@
 
     // create an object in new space
     Dart_Handle new_ref = Dart_NewString("new string");
     EXPECT_VALID(new_ref);
 
     // create an object in old space
     Dart_Handle old_ref;
     {
       Isolate* isolate = Isolate::Current();
       DARTSCOPE(isolate);
-      const String& str =
-          String::Handle(String::New("old string", Heap::kOld));
-      old_ref = Api::NewLocalHandle(isolate, str);
+      old_ref = Api::NewHandle(isolate, String::New("old string", Heap::kOld));
       EXPECT_VALID(old_ref);
     }
 
     // create a weak ref to the new space object
     weak_new_ref = Dart_NewWeakPersistentHandle(new_ref, NULL, NULL);
     EXPECT_VALID(weak_new_ref);
     EXPECT(!Dart_IsNull(weak_new_ref));
 
     // create a weak ref to the old space object
     weak_old_ref = Dart_NewWeakPersistentHandle(old_ref, NULL, NULL);
@@ skipping 114 matching lines @@
   Dart_Handle weak3 = Dart_Null();
   EXPECT(Dart_IsNull(weak3));
 
   Dart_Handle weak4 = Dart_Null();
   EXPECT(Dart_IsNull(weak4));
 
   Dart_EnterScope();
   {
     Isolate* isolate = Isolate::Current();
     DARTSCOPE(isolate);
-    String& str = String::Handle();
 
-    str ^= String::New("strongly reachable", Heap::kOld);
-    strong = Dart_NewPersistentHandle(Api::NewLocalHandle(isolate, str));
+    strong = Dart_NewPersistentHandle(
+        Api::NewHandle(isolate, String::New("strongly reachable", Heap::kOld)));
     EXPECT_VALID(strong);
     EXPECT(!Dart_IsNull(strong));
 
-    str ^= String::New("weakly reachable 1", Heap::kOld);
     weak1 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 1", Heap::kOld)),
+        NULL, NULL);
     EXPECT_VALID(weak1);
     EXPECT(!Dart_IsNull(weak1));
 
-    str ^= String::New("weakly reachable 2", Heap::kOld);
     weak2 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 2", Heap::kOld)),
+        NULL, NULL);
     EXPECT_VALID(weak2);
     EXPECT(!Dart_IsNull(weak2));
 
-    str ^= String::New("weakly reachable 3", Heap::kOld);
     weak3 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 3", Heap::kOld)),
+        NULL, NULL);
     EXPECT_VALID(weak3);
     EXPECT(!Dart_IsNull(weak3));
 
-    str ^= String::New("weakly reachable 4", Heap::kOld);
     weak4 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 4", Heap::kOld)),
+        NULL, NULL);
     EXPECT_VALID(weak4);
     EXPECT(!Dart_IsNull(weak4));
   }
   Dart_ExitScope();
 
   EXPECT_VALID(strong);
 
   EXPECT_VALID(weak1);
   EXPECT_VALID(weak2);
   EXPECT_VALID(weak3);
@@ skipping 135 matching lines @@
 
 TEST_CASE(PrologueWeakPersistentHandles) {
   Dart_Handle old_pwph = Dart_Null();
   EXPECT(Dart_IsNull(old_pwph));
   Dart_Handle new_pwph = Dart_Null();
   EXPECT(Dart_IsNull(new_pwph));
   Dart_EnterScope();
   {
     Isolate* isolate = Isolate::Current();
     DARTSCOPE(isolate);
-    String& str = String::Handle();
-    str ^= String::New("new space prologue weak", Heap::kNew);
     new_pwph = Dart_NewPrologueWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate,
+                       String::New("new space prologue weak", Heap::kNew)),
+        NULL, NULL);
     EXPECT_VALID(new_pwph);
     EXPECT(!Dart_IsNull(new_pwph));
-    str ^= String::New("old space prologue weak", Heap::kOld);
     old_pwph = Dart_NewPrologueWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate,
+                       String::New("old space prologue weak", Heap::kOld)),
+        NULL, NULL);
     EXPECT_VALID(old_pwph);
     EXPECT(!Dart_IsNull(old_pwph));
-    str ^= String::null();
   }
   Dart_ExitScope();
   EXPECT_VALID(new_pwph);
   EXPECT(!Dart_IsNull(new_pwph));
   EXPECT(Dart_IsPrologueWeakPersistentHandle(new_pwph));
   EXPECT_VALID(old_pwph);
   EXPECT(!Dart_IsNull(old_pwph));
   EXPECT(Dart_IsPrologueWeakPersistentHandle(old_pwph));
   // Garbage collect new space without invoking API callbacks.
   Isolate::Current()->heap()->CollectGarbage(Heap::kNew,
@@ skipping 34 matching lines @@
   EXPECT(Dart_IsNull(weak2));
 
   Dart_Handle weak3 = Dart_Null();
   EXPECT(Dart_IsNull(weak3));
 
 
   Dart_EnterScope();
   {
     Isolate* isolate = Isolate::Current();
     DARTSCOPE(isolate);
-    String& str = String::Handle();
 
-    str ^= String::New("strongly reachable", Heap::kOld);
-    strong = Dart_NewPersistentHandle(Api::NewLocalHandle(isolate, str));
+    strong = Dart_NewPersistentHandle(
+        Api::NewHandle(isolate, String::New("strongly reachable", Heap::kOld)));
     EXPECT(!Dart_IsNull(strong));
     EXPECT_VALID(strong);
 
-    str ^= String::New("weakly reachable 1", Heap::kOld);
     weak1 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 1", Heap::kOld)),
+        NULL, NULL);
     EXPECT(!Dart_IsNull(weak1));
     EXPECT_VALID(weak1);
 
-    str ^= String::New("weakly reachable 2", Heap::kOld);
     weak2 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 2", Heap::kOld)),
+        NULL, NULL);
     EXPECT(!Dart_IsNull(weak2));
     EXPECT_VALID(weak2);
 
-    str ^= String::New("weakly reachable 3", Heap::kOld);
     weak3 = Dart_NewWeakPersistentHandle(
-        Api::NewLocalHandle(isolate, str), NULL, NULL);
+        Api::NewHandle(isolate, String::New("weakly reachable 3", Heap::kOld)),
+        NULL, NULL);
     EXPECT(!Dart_IsNull(weak3));
     EXPECT_VALID(weak3);
   }
   Dart_ExitScope();
 
   EXPECT_VALID(strong);
 
   EXPECT_VALID(weak1);
   EXPECT_VALID(weak2);
   EXPECT_VALID(weak3);
@@ skipping 299 matching lines @@
   EXPECT(state != NULL);
   ApiLocalScope* scope = state->top_scope();
   Dart_Handle handles[300];
   {
     DARTSCOPE_NOCHECKS(isolate);
     Smi& val = Smi::Handle();
 
     // Start a new scope and allocate some local handles.
     Dart_EnterScope();
     for (int i = 0; i < 100; i++) {
-      val ^= Smi::New(i);
-      handles[i] = Api::NewLocalHandle(isolate, val);
+      handles[i] = Api::NewHandle(isolate, Smi::New(i));
     }
     EXPECT_EQ(100, state->CountLocalHandles());
     for (int i = 0; i < 100; i++) {
       val ^= Api::UnwrapHandle(handles[i]);
       EXPECT_EQ(i, val.Value());
     }
     // Start another scope and allocate some more local handles.
     {
       Dart_EnterScope();
       for (int i = 100; i < 200; i++) {
-        val ^= Smi::New(i);
-        handles[i] = Api::NewLocalHandle(isolate, val);
+        handles[i] = Api::NewHandle(isolate, Smi::New(i));
       }
       EXPECT_EQ(200, state->CountLocalHandles());
       for (int i = 100; i < 200; i++) {
         val ^= Api::UnwrapHandle(handles[i]);
         EXPECT_EQ(i, val.Value());
       }
 
       // Start another scope and allocate some more local handles.
       {
         Dart_EnterScope();
         for (int i = 200; i < 300; i++) {
-          val ^= Smi::New(i);
-          handles[i] = Api::NewLocalHandle(isolate, val);
+          handles[i] = Api::NewHandle(isolate, Smi::New(i));
         }
         EXPECT_EQ(300, state->CountLocalHandles());
         for (int i = 200; i < 300; i++) {
           val ^= Api::UnwrapHandle(handles[i]);
           EXPECT_EQ(i, val.Value());
         }
         EXPECT_EQ(300, state->CountLocalHandles());
         VERIFY_ON_TRANSITION;
         Dart_ExitScope();
       }
@@ skipping 1512 matching lines @@
   EXPECT(Dart_IsError(result));
 }
 
 
 TEST_CASE(NullReceiver) {
   Isolate* isolate = Isolate::Current();
   DARTSCOPE_NOCHECKS(isolate);
 
   Dart_Handle function_name = Dart_NewString("toString");
   const int number_of_arguments = 0;
-  Dart_Handle null_receiver = Api::NewLocalHandle(isolate, Object::Handle());
-  Dart_Handle result = Dart_InvokeDynamic(null_receiver,
+  Dart_Handle result = Dart_InvokeDynamic(Dart_Null(),
                                           function_name,
                                           number_of_arguments,
                                           NULL);
   EXPECT_VALID(result);
   EXPECT(Dart_IsString(result));
 
   // Should throw a NullPointerException. Disabled due to bug 5415268.
   /*
     Dart_Handle function_name2 = Dart_NewString("NoNoNo");
     result = Dart_InvokeDynamic(null_receiver,
@@ skipping 1292 matching lines @@
   EXPECT_VALID(result);
   EXPECT(Dart_IsDouble(result));
   double out;
   result = Dart_DoubleValue(result, &out);
   fprintf(stderr, "Benchmark_UseDartApi: %f us per iteration\n", out);
 }
 
 #endif  // defined(TARGET_ARCH_IA32) || defined(TARGET_ARCH_X64).
 
 }  // namespace dart