Resolution for issue #5092: Unit test handle checks consider dangling handles to be valid.

runtime/vm/handles_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 "platform/assert.h"
 #include "vm/flags.h"
 #include "vm/handles.h"
 #include "vm/heap.h"
 #include "vm/object.h"
 #include "vm/unit_test.h"
@@ skipping 59 matching lines @@
       const Smi& handle = Smi::Handle(Smi::New(i));
       EXPECT(handle.IsSmi());
       EXPECT_EQ(i, handle.Value());
     }
     EXPECT_EQ((handle_count + (2 * kNumHandles)),
               VMHandles::ScopedHandleCount());
   }
   EXPECT_EQ(handle_count, VMHandles::ScopedHandleCount());
 }
 
+
+// Unit test for Zone handle allocation.
+TEST_CASE(CheckHandleValidity) {
+#if defined(DEBUG)
+  FLAG_trace_handles = true;
+#endif
+  Dart_Handle handle = NULL;
+  {
+    StackZone sz(Thread::Current());
+    handle = reinterpret_cast<Dart_Handle>(&Smi::ZoneHandle(Smi::New(1)));
+    EXPECT_VALID(handle);
+  }
+  EXPECT_INVALID(handle);

[siva, 2017/01/18 23:27:06]

EXPECT(!Api::IsValid(handle));

[bkonyi, 2017/01/19 01:05:13]

Done.

+}
+
 }  // namespace dart