Revert "fix js2rt to not convert js promise into rt object (segfault) (#1879)"

This reverts commit eb5c207.

Author: mfiess

src/rtObject.cpp

@@ -158,8 +158,6 @@ rtError rtEmit::Send(int numArgs, const rtValue* args, rtValue* result)
         //
         // pass NULL as final argument for indication of asynchronous call
         err = e.f->Send(numArgs-1, args+1, &discard);
-        if (result)
-          *result = discard;
 #else
 
 #warning "  >>>>>>  No SYNC EVENTS... stopPropagation() will be broken !!"

src/rtScriptV8/rtWrapperUtils.cpp

@@ -497,13 +497,8 @@ rtValue js2rt(v8::Local<v8::Context>& ctx, const Handle<Value>& val, rtWrapperEr
     // It's very possible that someone is trying to use another wrapped object
     // from some other native addon. Maybe that would actuall work and fail
     // at a lower level?
-
     Local<Object> obj = val->ToObject();
-
-    if (obj->InternalFieldCount() > 0
-      && !val->IsArrayBufferView() // JS Buffer
-      && !val->IsPromise() // JS Promise
-      )
+    if (obj->InternalFieldCount() > 0 && !val->IsArrayBufferView() /*Buffer*/)
     {
       return rtObjectWrapper::unwrapObject(obj);
     }

tests/pxScene2d/CMakeLists.txt

@@ -162,7 +162,7 @@ set(TEST_SOURCE_FILES pxscene2dtestsmain.cpp  test_rtThreadPool.cpp test_example
     test_rtUrlUtils.cpp test_pxArchive.cpp test_pxPixel_h.cpp test_pxFont.cpp test_utf8.cpp
     test_rtSettings.cpp test_cors.cpp  test_external.cpp test_pxScene2d.cpp test_oscillate.cpp test_rtPathUtils.cpp
     test_rtError.cpp test_import_resources.cpp test_rtHttpRequest.cpp test_rtHttpResponse.cpp test_imagecacheSVG.cpp test_rtObjectWrapper.cpp
-    test_baseFilePath.cpp test_js2rt.cpp ${TEST_WAYLAND_SOURCE_FILES})
+    test_baseFilePath.cpp ${TEST_WAYLAND_SOURCE_FILES})
 
 if (DEFINED ENV{USE_HTTP_CACHE})
     message("Include http cache tests")