Skip to content
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

Render android #308

Merged
merged 2 commits into from Sep 16, 2019
Merged

Render android #308

merged 2 commits into from Sep 16, 2019

Conversation

ceyusa
Copy link
Contributor

@ceyusa ceyusa commented Sep 12, 2019

Add GL render for android target, which is a unix type with only EGL.

Since they are already especified when loaded by platform in the
GStreamerRender object.
@ceyusa ceyusa force-pushed the render-android branch 6 times, most recently from 5bab635 to 3047ced Compare September 12, 2019 12:59
@ceyusa
Copy link
Contributor Author

ceyusa commented Sep 12, 2019

Sadly, we don't have a test for android rendering gl textures. We need to test it in Servo directly.

@ferjm
Copy link
Contributor

ferjm commented Sep 12, 2019

I'll test it with Servo as soon as possible

@ceyusa
Copy link
Contributor Author

ceyusa commented Sep 16, 2019

removed the GL context activation

@ceyusa
Copy link
Contributor Author

ceyusa commented Sep 16, 2019

Can we merge this for now? @ferjm, @asajeffrey ?

@ferjm
Copy link
Contributor

ferjm commented Sep 16, 2019

Yes, I think we can merge as soon as CI is happy.

@asajeffrey
Copy link
Member

Fine by me. it would be nice if we could figure out a way to use features rather than gating on target_os, to reduce the amount of code duplication, but that's an orthogonal issue.

@ferjm
Copy link
Contributor

ferjm commented Sep 16, 2019

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 9add8b7 has been approved by ferjm

@bors-servo
Copy link
Contributor

⌛ Testing commit 9add8b7 with merge d6f799e...

bors-servo pushed a commit that referenced this pull request Sep 16, 2019
Render android

Add GL render for android target, which is a unix type with only EGL.
@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis
Approved by: ferjm
Pushing d6f799e to master...

@bors-servo bors-servo merged commit 9add8b7 into servo:master Sep 16, 2019
@ferjm ferjm deleted the render-android branch September 16, 2019 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants