Merge pull request #100 from pinepain/bump-v8

Bump v8 version

Author: pinepain

.travis.yml

@@ -6,7 +6,7 @@ services:
 
 env:
   global:
-    - V8=6.5.144
+    - V8=6.6.313
   matrix:
     - PHP=7.2
     - PHP=7.2 TEST_PHP_ARGS=-m

CMakeLists.txt

@@ -3,8 +3,8 @@ project(php-v8)
 
 # NOTE: This CMake file is just for syntax highlighting in CLion
 
-include_directories(/usr/local/opt/v8@6.5/include)
-include_directories(/usr/local/opt/v8@6.5/include/libplatform)
+include_directories(/usr/local/opt/v8@6.6/include)
+include_directories(/usr/local/opt/v8@6.6/include/libplatform)
 
 include_directories(/usr/local/include/php)
 include_directories(/usr/local/include/php/TSRM)

README.md

@@ -100,7 +100,7 @@ on top of php-v8 extension and makes embedding JavaScript in PHP easier.
 ### Requirements
 
 #### V8
-You will need a recent v8 Google JavaScript engine version installed. At this time v8 >= 6.5.144 required.
+You will need a recent v8 Google JavaScript engine version installed. At this time v8 >= 6.6.313 required.
 
 #### PHP
 This extension is PHP7-only. It works and tested with both PHP 7.0 and PHP 7.1.

config.m4

@@ -19,7 +19,7 @@ if test "$PHP_V8" != "no"; then
   SEARCH_PATH="/usr/local /usr"
   SEARCH_FOR="include/v8.h"
 
-  V8_MIN_API_VERSION_STR=6.5.144
+  V8_MIN_API_VERSION_STR=6.6.313
 
   DESIRED_V8_VERSION=`echo "${V8_MIN_API_VERSION_STR}" | $AWK 'BEGIN { FS = "."; } { printf "%s.%s", [$]1, [$]2;}'`
 
@@ -204,6 +204,7 @@ if test "$PHP_V8" != "no"; then
     src/php_v8_date.cc                                    \
     src/php_v8_regexp.cc                                  \
     src/php_v8_promise.cc                                 \
+    src/php_v8_promise_resolver.cc                        \
     src/php_v8_proxy.cc                                   \
     src/php_v8_number_object.cc                           \
     src/php_v8_boolean_object.cc                          \

scripts/provision/provision.sh

@@ -5,7 +5,7 @@ echo Provisioning...
 # Add Ondřej Surý's PPA with co-installable PHP versions:
 sudo add-apt-repository -y ppa:ondrej/php
 # Add libv8 PPA:
-sudo add-apt-repository ppa:pinepain/libv8-6.5
+sudo add-apt-repository ppa:pinepain/libv8-6.6
 
 # Let's update packages list:
 sudo apt-get update
@@ -19,7 +19,7 @@ sudo apt-get install -y git htop curl pkgconf
 
 
 # Build and development requirements
-sudo apt-get install -y libv8-6.5 libv8-6.5-dev libv8-6.5-dbg
+sudo apt-get install -y libv8-6.6 libv8-6.6-dev libv8-6.6-dbg
 sudo apt-get install -y dh-make valgrind
 sudo apt-get install -y libssl-dev openssl
 sudo apt-get install -y php7.2 php7.2-cli php7.2-dev php7.2-fpm

scripts/test_v8/hello_world.cpp

@@ -14,7 +14,7 @@ void weak_callback(const v8::WeakCallbackInfo<v8::Persistent<v8::String>>& data)
 
 int main(int argc, char* argv[]) {
   // Initialize V8.
-  v8::V8::InitializeICU();
+  //v8::V8::InitializeICU();
 
   std::unique_ptr<v8::Platform> platform = v8::platform::NewDefaultPlatform();
   v8::V8::InitializePlatform(platform.get());

scripts/test_v8/hello_world_build_deb.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
 
-ROOT=/opt/libv8-6.5
+ROOT=/opt/libv8-6.6
 LIB_DIR=$ROOT/lib/
 
 SRC_DIR=$ROOT

scripts/test_v8/hello_world_build_osx.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
 
-ROOT=/usr/local/opt/v8@6.5
+ROOT=/usr/local/opt/v8@6.6
 LIB_DIR=$ROOT/lib/
 
 SRC_DIR=$ROOT

src/php_v8_exceptions.h

@@ -81,8 +81,15 @@ extern void php_v8_throw_try_catch_exception(php_v8_context_t *php_v8_context, v
         return; \
     }
 
+#define PHP_V8_THROW_EXCEPTION_WHEN_UNDEFINED_CE(value, message, ce) \
+    if ((value)->IsUndefined()) { \
+        PHP_V8_THROW_EXCEPTION_CE(message, ce); \
+        return; \
+    }
+
 #define PHP_V8_THROW_VALUE_EXCEPTION_WHEN_NOTHING(value, message) PHP_V8_THROW_EXCEPTION_WHEN_NOTHING_CE((value), (message), php_v8_value_exception_class_entry)
 #define PHP_V8_THROW_VALUE_EXCEPTION_WHEN_EMPTY(value, message) PHP_V8_THROW_EXCEPTION_WHEN_EMPTY_CE((value), (message), php_v8_value_exception_class_entry)
+#define PHP_V8_THROW_VALUE_EXCEPTION_WHEN_UNDEFINED(value, message) PHP_V8_THROW_EXCEPTION_WHEN_UNDEFINED_CE((value), (message), php_v8_value_exception_class_entry)
 
 
 #define PHP_V8_THROW_EXCEPTION_WHEN_NOTHING(value, message) \
@@ -97,6 +104,7 @@ extern void php_v8_throw_try_catch_exception(php_v8_context_t *php_v8_context, v
         return; \
     }
 
+
 #define PHP_V8_THROW_EXCEPTION_WHEN_LIMITS_HIT(php_v8_context) \
     if ((php_v8_context)->php_v8_isolate->limits.time_limit_hit || (php_v8_context)->php_v8_isolate->limits.memory_limit_hit) { \
         php_v8_throw_try_catch_exception((php_v8_context), NULL); \

src/php_v8_object.cc

@@ -614,14 +614,17 @@ static PHP_METHOD(Object, getPropertyNames) {
     zend_long mode            = static_cast<zend_long>(v8::KeyCollectionMode::kOwnOnly);
     zend_long property_filter = static_cast<zend_long>(v8::PropertyFilter::ALL_PROPERTIES);
     zend_long index_filter    = static_cast<zend_long>(v8::IndexFilter::kIncludeIndices);
+    zend_bool convert_to_strings  = '\0';
 
-    if (zend_parse_parameters(ZEND_NUM_ARGS(), "o|lll", &context_zv, &mode, &property_filter, &index_filter) == FAILURE) {
+    if (zend_parse_parameters(ZEND_NUM_ARGS(), "o|lllb",
+                              &context_zv, &mode, &property_filter, &index_filter, &convert_to_strings) == FAILURE) {
         return;
     }
 
     mode = mode ? mode & PHP_V8_KEY_COLLECTION_MODE_FLAGS : mode;
     property_filter = property_filter ? property_filter & PHP_V8_PROPERTY_FILTER_FLAGS : property_filter;
     index_filter = index_filter ? index_filter & PHP_V8_INDEX_FILTER_FLAGS : index_filter;
+    v8::KeyConversionMode key_conversion = convert_to_strings ? v8::KeyConversionMode::kConvertToString : v8::KeyConversionMode::kKeepNumbers;
 
     PHP_V8_VALUE_FETCH_WITH_CHECK(getThis(), php_v8_value);
     PHP_V8_CONTEXT_FETCH_WITH_CHECK(context_zv, php_v8_context);
@@ -639,7 +642,8 @@ static PHP_METHOD(Object, getPropertyNames) {
     v8::MaybeLocal<v8::Array> maybe_local_array = local_object->GetPropertyNames(context,
                                                                                  static_cast<v8::KeyCollectionMode>(mode),
                                                                                  static_cast<v8::PropertyFilter >(property_filter),
-                                                                                 static_cast<v8::IndexFilter>(index_filter));
+                                                                                 static_cast<v8::IndexFilter>(index_filter),
+                                                                                 key_conversion);
 
     PHP_V8_MAYBE_CATCH(php_v8_context, try_catch);
     PHP_V8_THROW_EXCEPTION_WHEN_EMPTY(maybe_local_array, "Failed to get property names")
@@ -652,11 +656,13 @@ static PHP_METHOD(Object, getPropertyNames) {
 static PHP_METHOD(Object, getOwnPropertyNames) {
     zval *context_zv;
     zend_long filter = static_cast<zend_long>(v8::PropertyFilter::ALL_PROPERTIES);
+    zend_bool convert_to_strings  = '\0';
 
-    if (zend_parse_parameters(ZEND_NUM_ARGS(), "o|l", &context_zv, &filter) == FAILURE) {
+    if (zend_parse_parameters(ZEND_NUM_ARGS(), "o|lb", &context_zv, &filter, &convert_to_strings) == FAILURE) {
         return;
     }
     filter = filter ? filter & PHP_V8_PROPERTY_FILTER_FLAGS : filter;
+    v8::KeyConversionMode key_conversion = convert_to_strings ? v8::KeyConversionMode::kConvertToString : v8::KeyConversionMode::kKeepNumbers;
 
     PHP_V8_VALUE_FETCH_WITH_CHECK(getThis(), php_v8_value);
     PHP_V8_CONTEXT_FETCH_WITH_CHECK(context_zv, php_v8_context);
@@ -671,7 +677,7 @@ static PHP_METHOD(Object, getOwnPropertyNames) {
     PHP_V8_TRY_CATCH(isolate);
     PHP_V8_INIT_ISOLATE_LIMITS_ON_OBJECT_VALUE(php_v8_value);
 
-    v8::MaybeLocal<v8::Array> maybe_local_array = local_object->GetOwnPropertyNames(context, static_cast<v8::PropertyFilter >(filter));
+    v8::MaybeLocal<v8::Array> maybe_local_array = local_object->GetOwnPropertyNames(context, static_cast<v8::PropertyFilter >(filter), key_conversion);
 
     PHP_V8_MAYBE_CATCH(php_v8_context, try_catch);
     PHP_V8_THROW_EXCEPTION_WHEN_EMPTY(maybe_local_array, "Failed to get own property names")
@@ -1372,11 +1378,13 @@ PHP_V8_ZEND_BEGIN_ARG_WITH_RETURN_OBJ_INFO_EX(arginfo_getPropertyNames, ZEND_RET
                 ZEND_ARG_TYPE_INFO(0, mode, IS_LONG, 0)
                 ZEND_ARG_TYPE_INFO(0, property_filter, IS_LONG, 0)
                 ZEND_ARG_TYPE_INFO(0, index_filter, IS_LONG, 0)
+                ZEND_ARG_TYPE_INFO(0, convert_to_strings, _IS_BOOL, 0)
 ZEND_END_ARG_INFO()
 
 PHP_V8_ZEND_BEGIN_ARG_WITH_RETURN_OBJ_INFO_EX(arginfo_getOwnPropertyNames, ZEND_RETURN_VALUE, 1, V8\\ArrayObject, 0)
                 ZEND_ARG_OBJ_INFO(0, context, V8\\Context, 0)
                 ZEND_ARG_TYPE_INFO(0, filter, IS_LONG, 0)
+                ZEND_ARG_TYPE_INFO(0, convert_to_strings, _IS_BOOL, 0)
 ZEND_END_ARG_INFO()
 
 PHP_V8_ZEND_BEGIN_ARG_WITH_RETURN_OBJ_INFO_EX(arginfo_getPrototype, ZEND_RETURN_VALUE, 0, V8\\Value, 0)