New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 811962 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Feature

Blocking:
issue 732555
issue 811964



Sign in to add a comment

Re-Factor Android CompositorImpl’s GpuProcessTransportFactory-like Functionality

Project Member Reported by ericrk@chromium.org, Feb 13 2018

Issue description

For Viz to be enabled on Android, we need to add Viz-process setup code to the browser proc. On non-Android platforms this is handled by Swapping GpuProcessTransportFactory for a VizProcessTransportFactory. Unfortunately, Android doesn't use these classes.

Instead, the Android functionality similar to GpuProcessTransportFactory is embedded in CompositorImpl, making it hard to swap this logic out for a Viz version when in Viz mode. We should pull out the relevant code into a helper class / interface. We *may* be able to share some code with GpuProcessTransportFactory, but given the generally simpler Android requirements and the reliance of GpuProcessTransportFactory on ui::Compositor, this may not make sense.

 

Comment 1 by ericrk@chromium.org, Feb 13 2018

Blocking: 811964
Cc: samans@chromium.org

Sign in to add a comment