MojoGpuMemoryBufferManager should use mojom::GpuService::CreateGpuMemoryBuffer for allocating gpu memory.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0f4b93cef1da595d51b79c84e3ce1601e4efab4c commit 0f4b93cef1da595d51b79c84e3ce1601e4efab4c Author: sadrul <sadrul@chromium.org> Date: Wed Aug 17 17:55:45 2016 mojo: Struct traits for gpu buffer types. BUG= 637923 TBR=sky@ for trivial services/ui/gpu update. Review-Url: https://codereview.chromium.org/2248973005 Cr-Commit-Position: refs/heads/master@{#412581} [modify] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/services/ui/gpu/gpu_service_impl.cc [modify] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/services/ui/gpu/gpu_service_impl.h [modify] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/services/ui/public/interfaces/gpu_memory_buffer.mojom [modify] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/services/ui/public/interfaces/gpu_service.mojom [modify] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/ui/gfx/mojo/BUILD.gn [add] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/ui/gfx/mojo/buffer_types.mojom [add] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/ui/gfx/mojo/buffer_types.typemap [add] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/ui/gfx/mojo/buffer_types_enum_traits.h [modify] https://crrev.com/0f4b93cef1da595d51b79c84e3ce1601e4efab4c/ui/gfx/typemaps.gni
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/900faab623165140c2c34f7ba94cced55769691c commit 900faab623165140c2c34f7ba94cced55769691c Author: sadrul <sadrul@chromium.org> Date: Wed Aug 31 00:40:02 2016 gfx: Struct traits for GpuMemoryBufferId and GpuMemoryBufferType. BUG= 637923 Review-Url: https://codereview.chromium.org/2295923002 Cr-Commit-Position: refs/heads/master@{#415521} [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/common/gpu_type_converters.cc [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/common/gpu_type_converters.h [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/common/gpu_type_converters_unittest.cc [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/gpu/interfaces/gpu_service_internal.mojom [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/public/interfaces/gpu_memory_buffer.mojom [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/public/interfaces/gpu_service.mojom [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/ws/gpu_service_proxy.cc [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/services/ui/ws/gpu_service_proxy.h [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/ui/gfx/mojo/buffer_types.mojom [modify] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/ui/gfx/mojo/buffer_types.typemap [rename] https://crrev.com/900faab623165140c2c34f7ba94cced55769691c/ui/gfx/mojo/buffer_types_traits.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5569cc1d0d05edce033dc92096ad5d5b03067907 commit 5569cc1d0d05edce033dc92096ad5d5b03067907 Author: sadrul <sadrul@chromium.org> Date: Fri Sep 02 22:09:15 2016 gfx: Struct traits for GpuMemoryBufferHandle. The remaining type-converter is for GpuMemoryBufferHandle. So introduce a struct-trait for it instead. It also needs struct-traits for NativePixmapPlane and NativePixmapHandle. Remove the type-converters, since they are no longer needed. Also, expose NativePixmapPlane and NativePixmapHandle types to non-ozone platforms, so we don't need some special ozone-only mojom API. BUG= 637923 TBR=reveman@ for trivial components/exo change Review-Url: https://codereview.chromium.org/2298003002 Cr-Commit-Position: refs/heads/master@{#416362} [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/components/exo/display.h [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/services/ui/common/BUILD.gn [delete] https://crrev.com/60fb6b0f601074d94dc03bbea9f1351f678f18fa/services/ui/common/gpu_type_converters.cc [delete] https://crrev.com/60fb6b0f601074d94dc03bbea9f1351f678f18fa/services/ui/common/gpu_type_converters.h [delete] https://crrev.com/60fb6b0f601074d94dc03bbea9f1351f678f18fa/services/ui/common/gpu_type_converters_unittest.cc [delete] https://crrev.com/60fb6b0f601074d94dc03bbea9f1351f678f18fa/services/ui/common/mus_common_unittests_app_manifest.json [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/services/ui/public/cpp/gpu_service.cc [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/services/ui/public/interfaces/gpu_service.mojom [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/services/ui/ws/gpu_service_proxy.cc [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/tools/determinism/compare_build_artifacts.py [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/BUILD.gn [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/gpu_memory_buffer.h [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/ipc/gfx_param_traits_macros.h [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/mojo/buffer_types.mojom [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/mojo/buffer_types.typemap [add] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/mojo/buffer_types_traits.cc [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/mojo/buffer_types_traits.h [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/mojo/struct_traits_unittest.cc [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/mojo/traits_test_service.mojom [rename] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/native_pixmap_handle.cc [rename] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/gfx/native_pixmap_handle.h [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/ozone/platform/drm/client_native_pixmap_factory_gbm.cc [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/ozone/platform/drm/gpu/drm_thread.h [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/ozone/platform/drm/gpu/gbm_buffer.cc [modify] https://crrev.com/5569cc1d0d05edce033dc92096ad5d5b03067907/ui/ozone/public/native_pixmap.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/763bd5900392427eff240119f238754c1e4923e1 commit 763bd5900392427eff240119f238754c1e4923e1 Author: sadrul <sadrul@chromium.org> Date: Tue Sep 27 14:33:44 2016 services/ui: Add mojom API for creating/destroying gpu memory buffer. Add struct-traits for gpu::SurfaceHandle, and use that to provide mojom API for creating/destroying gpu memory buffer. This allows removing the direct dependency of gpu code from ws code (which is a pre-requisite for the gpu process split). BUG= 637923 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2360203003 Cr-Commit-Position: refs/heads/master@{#421201} [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/components/display_compositor/BUILD.gn [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/command_buffer/client/BUILD.gn [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/command_buffer/service/BUILD.gn [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/ipc/common/BUILD.gn [add] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/ipc/common/surface_handle.mojom [add] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/ipc/common/surface_handle.typemap [add] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/ipc/common/surface_handle_struct_traits.h [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/gpu/ipc/common/typemaps.gni [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/gpu/gpu_service_internal.cc [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/gpu/gpu_service_internal.h [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/gpu/interfaces/gpu_service_internal.mojom [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/ws/DEPS [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/ws/gpu_service_proxy.cc [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/ws/mus_gpu_memory_buffer_manager.cc [modify] https://crrev.com/763bd5900392427eff240119f238754c1e4923e1/services/ui/ws/mus_gpu_memory_buffer_manager.h
Comment 1 by bugdroid1@chromium.org
, Aug 17 2016