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

Issue 756530 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

exo CompositorFrames are not the same size as contents

Project Member Reported by kylec...@chromium.org, Aug 17 2017

Issue description

The size of exo CompositorFrames doesn't match the size of the DrawQuad contents in the CompositorFrame. SurfaceTreeHost::SubmitCompositorFrame() always generates a CompositorFrame that is the same size as the display. 

It should be relatively simple to fix. We draw a border/shadow around the exo window, so we know the size and offset, which is all the information needed to do generate the correct CompositorFrame.
 
Labels: Proj-Mustash
Owner: penghuang@chromium.org
Status: Assigned (was: Available)
Cc: lpique@chromium.org
I remember David maintained it is because wayland doesn't allow set absolute position of a surface. So we create a shell surface with the size of the screen and make it transparent. And then put the content of the app as a sub surface on it.

To get ride of the display size shell surface, we need modify the code in android side.

David and Lloyd, could you please add more detail about it? Thanks.
#2, correct. There are some long term plans for moving Arc to something that is closer to a standard wayland app but not sure what the time line for that is.

The display sized surface at the bottom of each arc app is backed by a 1x1 buffer scaled to the display size and it has alpha=0 so we can probably detect that and limit the size of CompositorFrames based on it.
Labels: -Pri-2 Pri-1
This is potentially blocking surface sync and viz work. Can we figure out a way to ensure that the CompositorFrame size matches the window size in exo despite ARC++ weirdness? Marking as P1.

Comment 5 by sadrul@chromium.org, Aug 23 2017

Cc: sadrul@chromium.org
Labels: -Pri-1 Pri-3
Owner: ----
Status: Available (was: Assigned)
I think we already workaround this problem. So it doesn't block surface sync and viz work.

Comment 7 by laforge@google.com, Nov 8 2017

Components: -Internals>Viz Internals>Services>Viz
Migrating from Internals>Viz to Internals>Services>Viz.
Labels: -Proj-Mustash Proj-Mash-MultiProcess

Sign in to add a comment