-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[TIMOB-23903] Build against V8 5.7.492.71 (#8382)
* Use V8 5.3.332.41 * Move from WeakCallbackData to WeakCallbackInfo API, since the former is removed now. * Update calls to Template#Set(). Can only set primitives or other Templates, so the Set*Method calls needed to set the generated FunctionTemplate (and not it's function). Also need to hack around hanging an External wrapping a jclass on the __javaClass_ property by using a natiev data property getter callback that gets the External passed as Data and returning that. * Bump module api version for Andorid to 4 * Rebuild the ti.map and ti.facebook native modules for Android with higher version/moduleApiVersion against a build of the SDK * Add scons binary script * Getting closer to working, mocha tests crash in the middle not at startup now. * Hmm, now we crash with a reference table overflow in JNI for PlatformModule... * Closer still. Crashes on Ti.UI.Label.apiname test, because getproxyTemplate must be throwing some uncaught excpetion, so the toLocalChecked() on GetFunction is failing. * Fix double listing of property * Add exception handling for cases where we blindly called ToLocalChecked() before * Try to set up so when JVM collects a weak reference to a Java object with a proxy that we then go back and kill the proxy/JS object on the C++ side. Tehre's still an issue here where we're not cleaning up jclass references somehow that I need to fix... * Booya! Got it to stop crashing on the nasty Ti.UI.Window test * Fix duplicate property name errors on WebView proxy * Fix local ref leak, guard against trying to reset the Persistent twice in destructor * Be more paranoid about cleaning up local refs in Proxy for jobjects * Clean up docs for ReferenceTable * Start rolling back some debug stuff * Remove FIXME that no longer applies, lower log level of debug log * Remove FIXME that no longer applies * lower log level of debug log * Add note about possibly improving raw pointer usage across the JNI boundary to Proxy instances * Re-enable most of the XML tests now that we don't have JNI leaks so they don't *all* barf/crash * Roll back the reference queue/proxy cleanup stuff on ReferenceTable. General cleanup * use longs for keys in ReferenceTable, not ints. Clean up header includes. * USe non-deprecated version of v8::Debug::ProcessDebugMessages() * Update to V8 5.7.492.71 * Fix reference to V8Util to be titanium::V8Util
- Loading branch information
1 parent
14c4f4f
commit 193e13e
Showing
41 changed files
with
1,830 additions
and
486 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
libv8.version=5.1.281.59 | ||
libv8.version=5.7.492.71 | ||
libv8.mode=release |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,7 +16,6 @@ else | |
APP_ABI := armeabi-v7a | ||
endif | ||
|
||
|
||
TARGET_DEVICE := device | ||
APP_OPTIM := release | ||
TI_DEBUG := 1 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
78 changes: 60 additions & 18 deletions
78
android/runtime/v8/src/java/org/appcelerator/kroll/runtime/v8/ReferenceTable.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.