-
Notifications
You must be signed in to change notification settings - Fork 165
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
Fix UI Thread deadlock #644
Conversation
* Don't call getApplication().registerView( ) at application. * Move code from Activity to GVRf Main class * Fix layout * Use IViewEvents listener. GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
gvr-3dcursor doesn't compile for me. |
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
Hi @liaxim, I have fixed the 3dcursor issues. |
gvr-remote-scripting doesn't compile |
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
@liaxim Sorry, |
@@ -2,8 +2,9 @@ | |||
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android" | |||
android:layout_width="wrap_content" | |||
android:layout_height="wrap_content" | |||
android:background="@color/white_transparent" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get an aapt error: AAPT: error: resource color/white_transparent (aka org.gearvrf.renderableview:color/white_transparent) not found.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@liaxim fixed!
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes <ragner.n@samsung.com>
8d2d4f5
to
d7bb9ad
Compare
Looks good. Planning to merge on Friday. |
Fixes to UI thread deadlock and improvements to test Samsung/GearVRf#1973
GearVRf-DCO-1.0-Signed-off-by: Ragner Magalhaes ragner.n@samsung.com