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 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 ceyusa:render-android branch 6 times, most recently from 5bab635 to 3047ced Sep 12, 2019
@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
Member

ferjm commented Sep 12, 2019

I'll test it with Servo as soon as possible

@ceyusa ceyusa force-pushed the ceyusa:render-android branch from 3047ced to 6e60bca Sep 16, 2019
@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
Member

ferjm commented Sep 16, 2019

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

@asajeffrey
Copy link
Member

asajeffrey commented Sep 16, 2019

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.

Fixes #307
@ceyusa ceyusa force-pushed the ceyusa:render-android branch from 6e60bca to 9add8b7 Sep 16, 2019
@ferjm
Copy link
Member

ferjm commented Sep 16, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Sep 16, 2019

📌 Commit 9add8b7 has been approved by ferjm

@bors-servo
Copy link
Contributor

bors-servo commented Sep 16, 2019

Testing commit 9add8b7 with merge d6f799e...

bors-servo added 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

bors-servo commented Sep 16, 2019

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

@bors-servo bors-servo merged commit 9add8b7 into servo:master Sep 16, 2019
3 checks passed
3 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@ferjm ferjm deleted the ceyusa:render-android branch Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.