Merge lp://qastaging/~afrantzis/mir/texture-bindable into lp://qastaging/mir
Status: | Rejected |
---|---|
Rejected by: | Alexandros Frantzis |
Proposed branch: | lp://qastaging/~afrantzis/mir/texture-bindable |
Merge into: | lp://qastaging/mir |
Prerequisite: | lp://qastaging/~afrantzis/mir/buffer-native-type |
Diff against target: |
946 lines (+316/-55) 32 files modified
examples/CMakeLists.txt (+2/-0) examples/buffer_render_target.cpp (+7/-1) examples/server_example_adorning_compositor.cpp (+12/-1) include/platform/mir/graphics/buffer.h (+0/-1) src/platforms/CMakeLists.txt (+1/-0) src/platforms/android/server/buffer.h (+3/-1) src/platforms/mesa/server/common/gbm_buffer.h (+3/-1) src/platforms/mesa/server/common/shm_buffer.h (+3/-1) src/renderers/gl/include/mir/renderer/gl/texture_bindable.h (+46/-0) src/server/compositor/CMakeLists.txt (+4/-0) src/server/compositor/recently_used_cache.cpp (+11/-1) src/server/compositor/screencast_display_buffer.cpp (+8/-2) src/server/compositor/screencast_display_buffer.h (+2/-0) src/server/compositor/temporary_buffers.cpp (+0/-5) src/server/compositor/temporary_buffers.h (+0/-1) src/server/scene/CMakeLists.txt (+4/-0) src/server/scene/gl_pixel_buffer.cpp (+7/-1) tests/CMakeLists.txt (+1/-0) tests/include/mir/test/doubles/mock_buffer.h (+4/-1) tests/include/mir/test/doubles/mock_gl_buffer.h (+45/-0) tests/include/mir/test/doubles/stub_buffer.h (+0/-1) tests/include/mir/test/doubles/stub_gl_buffer.h (+45/-0) tests/include/mir/test/doubles/stub_gl_buffer_allocator.h (+45/-0) tests/integration-tests/graphics/mesa/test_buffer_integration.cpp (+12/-3) tests/integration-tests/test_session.cpp (+21/-1) tests/unit-tests/compositor/test_compositing_screencast.cpp (+6/-6) tests/unit-tests/compositor/test_gl_renderer.cpp (+3/-3) tests/unit-tests/compositor/test_gl_texture_cache.cpp (+4/-3) tests/unit-tests/compositor/test_screencast_display_buffer.cpp (+8/-8) tests/unit-tests/compositor/test_temporary_buffers.cpp (+0/-9) tests/unit-tests/graphics/mesa/kms/test_gbm_buffer.cpp (+7/-2) tests/unit-tests/scene/test_gl_pixel_buffer.cpp (+2/-2) |
To merge this branch: | bzr merge lp://qastaging/~afrantzis/mir/texture-bindable |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alan Griffiths | Needs Fixing | ||
Cemil Azizoglu (community) | Needs Information | ||
Chris Halse Rogers | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email:
|
Commit message
platform: Move gl_bind_to_texture method from Buffer to mir::renderer:
Description of the change
platform: Move gl_bind_to_texture method from Buffer to mir::renderer:
This MP makes the Buffer class agnostic of rendering technology, by moving the gl_bind_to_buffer() method to a renderer specific interface (TextureBindable). This interface is implemented by the native buffer types and accessed by components that need it by dynamically casting Buffer:
The definition of TextureBindable will eventually be made public (e.g. shipped in a mir-renderer-gl-dev package), so I have made it accessible from most of the non-core codebase (tests and examples). The core (libmirserver), however, should not be aware of the rendering-specific technologies, so I've tried to limit exposure to TextureBindable only to parts that currently need it (e.g. scene, compositor). These components will be eventually cleaned to remove this dependency.
Next steps involve isolating the GL dependencies from our core, so they can be moved into a different shared object (i.e. plugin).
Unmerged revisions
- 2881. By Alexandros Frantzis
-
platform: Move gl_bind_to_texture method from Buffer to mir::renderer:
:gl::TextureBin dable - 2880. By Alexandros Frantzis
-
platform: Introduce Buffer:
:native_ type() method
PASSED: Continuous integration, rev:2881 jenkins. qa.ubuntu. com/job/ mir-ci/ 4685/ jenkins. qa.ubuntu. com/job/ mir-android- vivid-i386- build/3683 s-jenkins. ubuntu- ci:8080/ job/mir- clang-ts- vivid-amd64- build/104 jenkins. qa.ubuntu. com/job/ mir-clang- vivid-amd64- build/2594 jenkins. qa.ubuntu. com/job/ mir-mediumtests -vivid- touch/3633 jenkins. qa.ubuntu. com/job/ mir-wily- amd64-ci/ 834 jenkins. qa.ubuntu. com/job/ mir-wily- amd64-ci/ 834/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ mir-mediumtests -builder- vivid-armhf/ 3633 jenkins. qa.ubuntu. com/job/ mir-mediumtests -builder- vivid-armhf/ 3633/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ mir-mediumtests -runner- mako/6402 s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 22846
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/mir- ci/4685/ rebuild
http://