New issue
Advanced search Search tips

Issue 905394 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 881574



Sign in to add a comment

Mash Creating Partial Window Tree Connections

Project Member Reported by jonr...@chromium.org, Nov 14

Issue description

Can reproduce with: ./out/cros/interactive_ui_tests --enable-features=Mash --gtest_filter=DownloadNotificationTest.IncognitoDownloadFile

In this test I see two ClientRoots being created. One is the top_level, while the other is not.

The non-top_level ClientRoot is set to not assign LocalSurfaceIds (it's ServerWindow has an owning tree. However the client never sends the Window Service a valid LocalSurfaceId. So it fails to embed.

The connection itself seems to only be partially complete.
  -  ClientRoot::HandleBoundsOrScaleFactorChange attempts to notify the WindowTreeClient, but that message is just being enqueued. No WindowTreeClient seems to receive it
  -  WindowTree::SetWindowBounds is being called by the client, though with an invalid LocalSurfaceId.

So it appears that the client is around, but that the connections are being fully setup. Any ideas?
 
The non-top level one is likely for a renderer. We want to get rid of the content code specific to mus, but it's blocked on some other things. See bug 881574.
Blockedon: 881574
Likely, I didn't debug into the source of the WindowTree::SetWindowBounds calls.

This shouldn't block the upcoming LocalSurfaceId changes I have planned. But good to know there are long term plans to address the root cause.

Sign in to add a comment