From: Adeel Kazmi Date: Fri, 10 Jul 2015 09:52:36 +0000 (-0700) Subject: Merge "Fixed crash while wrapping a Control as an actor in JavaScript plugin" into... X-Git-Tag: dali_1.0.49~23 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=456a3e43364ba389325170d8355384bbd3f6dd6c;hp=df3f8087cae4d35a0906884f144707a339bda072 Merge "Fixed crash while wrapping a Control as an actor in JavaScript plugin" into devel/master --- diff --git a/plugins/dali-script-v8/src/actors/actor-wrapper.cpp b/plugins/dali-script-v8/src/actors/actor-wrapper.cpp index 7b9b158..149edb9 100644 --- a/plugins/dali-script-v8/src/actors/actor-wrapper.cpp +++ b/plugins/dali-script-v8/src/actors/actor-wrapper.cpp @@ -20,6 +20,7 @@ // EXTERNAL INCLUDES #include +#include // INTERNAL INCLUDES #include @@ -318,7 +319,10 @@ ActorWrapper::ActorWrapper( Actor actor, v8::Handle ActorWrapper::WrapActor(v8::Isolate* isolate, Actor actor ) { v8::EscapableHandleScope handleScope( isolate ); - v8::Local object = WrapActor( isolate, actor, GetActorType( actor.GetTypeName() ) ); + + // Check whether the actor is a Control + ActorWrapper::ActorType type = Toolkit::Control::DownCast(actor) ? ACTOR : GetActorType( actor.GetTypeName() ); + v8::Local object = WrapActor( isolate, actor, type ); return handleScope.Escape( object ); } diff --git a/plugins/dali-script-v8/src/dali-wrapper.cpp b/plugins/dali-script-v8/src/dali-wrapper.cpp index c716f75..257b5d4 100644 --- a/plugins/dali-script-v8/src/dali-wrapper.cpp +++ b/plugins/dali-script-v8/src/dali-wrapper.cpp @@ -63,9 +63,7 @@ const ApiFunction ConstructorFunctionTable[]= { "PathConstrainer", PathConstrainerWrapper::NewPathConstrainer}, { "LinearConstrainer", LinearConstrainerWrapper::NewLinearConstrainer}, { "Actor", ActorWrapper::NewActor }, - { "TextActor", ActorWrapper::NewActor }, { "ImageActor", ActorWrapper::NewActor }, - { "MeshActor", ActorWrapper::NewActor }, { "CameraActor", ActorWrapper::NewActor }, { "Layer", ActorWrapper::NewActor }, { "Control", ActorWrapper::NewControl },