New issue
Advanced search Search tips

Issue 891064 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 1
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Mash: Chrome app window doesn't initially respond to input

Project Member Reported by est...@chromium.org, Oct 1

Issue description

1. launch chrome os with --enable-features=SingleProcessMash --load-extension=/chrome-app-samples/samples/window-options
2. start the window options app
3. observe that no clicks get through to the app
4. resize app window
5. clicks work

(the chrome app samples repo is https://github.com/GoogleChrome/chrome-app-samples )
 
Evan, is this still happening?
yep, I'm still seeing it as of 6f70af53a445db358606969 (r602939)
Owner: sky@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96

commit b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96
Author: Scott Violet <sky@chromium.org>
Date: Thu Nov 01 19:55:58 2018

window-service: Calls SetClientArea() on server when client-view changes

The client area is dictated by the bounds of the ClientView. This patch
makes DesktopWindowTreeHostMus observe bounds changes of the ClientView, not
the non-client-frame-view.

BUG= 891064 
TEST=covered by test

Change-Id: Ia96c2be5b9af1bce24d253e01dbf673accc20d3a
Reviewed-on: https://chromium-review.googlesource.com/c/1312482
Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604679}
[modify] https://crrev.com/b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96/ui/aura/test/mus/window_tree_client_private.cc
[modify] https://crrev.com/b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96/ui/aura/test/mus/window_tree_client_private.h
[modify] https://crrev.com/b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96/ui/views/mus/desktop_window_tree_host_mus.cc
[modify] https://crrev.com/b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96/ui/views/mus/desktop_window_tree_host_mus.h
[modify] https://crrev.com/b5129c9a6cb8cd052190d59a1c8c55b9be6e9e96/ui/views/mus/desktop_window_tree_host_mus_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment