Add Isolate::IsInUse() method

Author: pinepain

src/php_v8_isolate.cc

@@ -489,7 +489,6 @@ static PHP_METHOD(V8Isolate, SetCaptureStackTraceForUncaughtExceptions) {
                                                        static_cast<v8::StackTrace::StackTraceOptions>(options & PHP_V8_STACK_TRACE_OPTIONS));
 }
 
-
 static PHP_METHOD(V8Isolate, IsDead) {
     if (zend_parse_parameters_none() == FAILURE) {
         return;
@@ -501,6 +500,17 @@ static PHP_METHOD(V8Isolate, IsDead) {
     RETURN_BOOL(isolate->IsDead());
 }
 
+static PHP_METHOD(V8Isolate, IsInUse) {
+    if (zend_parse_parameters_none() == FAILURE) {
+        return;
+    }
+
+    PHP_V8_ISOLATE_FETCH_WITH_CHECK(getThis(), php_v8_isolate);
+    PHP_V8_DECLARE_ISOLATE(php_v8_isolate);
+
+    RETURN_BOOL(isolate->IsInUse());
+}
+
 
 ZEND_BEGIN_ARG_INFO_EX(arginfo_v8_isolate___construct, ZEND_SEND_BY_VAL, ZEND_RETURN_VALUE, 0)
                 ZEND_ARG_OBJ_INFO(0, snapshot, V8\\StartupData, 1)
@@ -574,6 +584,10 @@ ZEND_END_ARG_INFO()
 ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_v8_isolate_IsDead, ZEND_RETURN_VALUE, 0, _IS_BOOL, NULL, 0)
 ZEND_END_ARG_INFO()
 
+ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_v8_isolate_IsInUse, ZEND_RETURN_VALUE, 0, _IS_BOOL, NULL, 0)
+ZEND_END_ARG_INFO()
+
+
 static const zend_function_entry php_v8_isolate_methods[] = {
         PHP_ME(V8Isolate, __construct, arginfo_v8_isolate___construct, ZEND_ACC_PUBLIC | ZEND_ACC_CTOR)
 
@@ -603,6 +617,7 @@ static const zend_function_entry php_v8_isolate_methods[] = {
 
         PHP_ME(V8Isolate, SetCaptureStackTraceForUncaughtExceptions, arginfo_v8_isolate_SetCaptureStackTraceForUncaughtExceptions, ZEND_ACC_PUBLIC)
         PHP_ME(V8Isolate, IsDead, arginfo_v8_isolate_IsDead, ZEND_ACC_PUBLIC)
+        PHP_ME(V8Isolate, IsInUse, arginfo_v8_isolate_IsInUse, ZEND_ACC_PUBLIC)
 
         PHP_FE_END
 };

stubs/src/Isolate.php

@@ -196,11 +196,22 @@ public function LowMemoryNotification()
     /**
      * Check if V8 is dead and therefore unusable.  This is the case after
      * fatal errors such as out-of-memory situations.
+     *
+     * @return bool
      */
     public function IsDead() : bool
     {
     }
 
+    /**
+     * Check if this isolate is in use.
+     * True if at least one thread Enter'ed this isolate.
+     *
+     * @return bool
+     */
+    public function IsInUse() : bool
+    {
+    }
 
     /**
      * Tells V8 to capture current stack trace when uncaught exception occurs

tests/.testsuite.php

@@ -39,6 +39,12 @@ public function inline($message, $value)
         echo $message, ': ', $value, PHP_EOL;
     }
 
+    public function inline_dump($message, $value)
+    {
+        echo $message, ': ';
+        $this->dump($value);
+    }
+
     public function space($new_lines_number = 2)
     {
         echo str_repeat(PHP_EOL, max(1, $new_lines_number));

tests/V8Isolate.phpt

@@ -46,7 +46,7 @@ object(V8\Isolate)#2 (5) {
 V8\Exceptions\GenericException: Not in context!
 
 V8\Isolate->GetHeapStatistics():
-    object(V8\HeapStatistics)#28 (9) {
+    object(V8\HeapStatistics)#29 (9) {
       ["total_heap_size":"V8\HeapStatistics":private]=>
       float(%d)
       ["total_heap_size_executable":"V8\HeapStatistics":private]=>

tests/V8Isolate_IsInUse.phpt

@@ -0,0 +1,32 @@
+--TEST--
+V8\Isolate::IsInUse()
+--SKIPIF--
+<?php if (!extension_loaded("v8")) print "skip"; ?>
+--FILE--
+<?php
+
+/** @var \Phpv8Testsuite $helper */
+$helper = require '.testsuite.php';
+
+require '.v8-helpers.php';
+$v8_helper = new PhpV8Helpers($helper);
+
+
+$isolate = new V8\Isolate();
+
+$helper->inline_dump('Isolate in use', $isolate->IsInUse());
+
+$context = new V8\Context($isolate);
+$fnc = new \V8\FunctionObject($context, function (\V8\CallbackInfo $info) use ($helper) {
+  $helper->inline_dump('Isolate in use', $info->GetIsolate()->IsInUse());
+});
+
+
+$context->GlobalObject()->Set($context, new \V8\StringValue($isolate, 'test'), $fnc);
+
+(new \V8\Script($context, new \V8\StringValue($isolate, 'test()')))->Run($context);
+
+?>
+--EXPECT--
+Isolate in use: bool(false)
+Isolate in use: bool(true)