Skip to content

[weex] Adjust the weex platform entry file to fit the new weex runtime #6620

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Sep 18, 2017

Conversation

Hanks10100
Copy link
Contributor

These changes will only affect the Weex platform, and will not affect any features on the web.

Remove the new Function hack for V8

Remove the callFunctionNative method and the compileBundle API which is provided by modified V8.
In order to maintain the consistency of the js engine, Weex also uses JSC on Android. The legacy hack for V8 engine should be removed.

Move shared APIs to Weex runtime

Modules and components should be registered in weex runtime, not in each front-end frameworks. The
registerModules, registerComponents, weex.supports and weex.requireModule APIs is moved to
Weex runtime which is in the https://github.com/apache/incubator-weex .

Use document instead of renderer

Using document is more standard than using renderer. The createElement API support to create custom WeexElement now.

Test cases

Weex's test cases have also been adjusted to fit the new weex-js-runtime architecture.

yyx990803 and others added 5 commits August 30, 2017 00:54
Remove the `callFunctionNative` method and `compileBundle`, which is provided by modified V8. In
order to maintain the consistency of the js engine, Weex also use JSC on Android. The legacy hack
for V8 engine should be removed.
Modules and components should be registered in weex runtime, not the specific framework. The
`registerModules`, `registerComponents`, `weex.supports` and `weex.requireModule` api is moved to
weex runtime, which is in the "apache/incubator-weex" repo.
@Hanks10100 Hanks10100 force-pushed the weex-refactor branch 2 times, most recently from 65f9c19 to 2639d70 Compare September 16, 2017 12:53
@yyx990803 yyx990803 merged commit 2deda3d into vuejs:dev Sep 18, 2017
@Hanks10100 Hanks10100 deleted the weex-refactor branch September 19, 2017 09:21
ztlevi pushed a commit to ztlevi/vue that referenced this pull request Feb 14, 2018
…ex runtime (vuejs#6620)

* trim trailing whitespace

* revert(weex): remove the new Function hack for V8, as Weex uses JSC now

Remove the `callFunctionNative` method and `compileBundle`, which is provided by modified V8. In
order to maintain the consistency of the js engine, Weex also use JSC on Android. The legacy hack
for V8 engine should be removed.

* refactor(weex): move module and component apis to weex runtime

Modules and components should be registered in weex runtime, not the specific framework. The
`registerModules`, `registerComponents`, `weex.supports` and `weex.requireModule` api is moved to
weex runtime, which is in the "apache/incubator-weex" repo.

* test(weex): refactor the test cases to fit the new weex-js-runtime

* fix(weex): use document instead of renderer to create Element
f2009 pushed a commit to f2009/vue that referenced this pull request Jan 25, 2019
…ex runtime (vuejs#6620)

* trim trailing whitespace

* revert(weex): remove the new Function hack for V8, as Weex uses JSC now

Remove the `callFunctionNative` method and `compileBundle`, which is provided by modified V8. In
order to maintain the consistency of the js engine, Weex also use JSC on Android. The legacy hack
for V8 engine should be removed.

* refactor(weex): move module and component apis to weex runtime

Modules and components should be registered in weex runtime, not the specific framework. The
`registerModules`, `registerComponents`, `weex.supports` and `weex.requireModule` api is moved to
weex runtime, which is in the "apache/incubator-weex" repo.

* test(weex): refactor the test cases to fit the new weex-js-runtime

* fix(weex): use document instead of renderer to create Element
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants