running cts sometimes leave a system modal window focusable. |
|||||||||
Issue descriptionNormal task window shouldn't be in the system modal container. If if it does, other windows created by an app should be removed once app is removed. So far I've seen only in CTS. Filing the bug in case we see this in normal situation.
,
Aug 28
,
Aug 31
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5b1c668cf2ddc7418b79a61a7c38a0a3bec86e53 commit 5b1c668cf2ddc7418b79a61a7c38a0a3bec86e53 Author: Mitsuru Oshima <oshima@chromium.org> Date: Fri Aug 31 21:38:35 2018 Don't allow window frame on shell surfce in system container Normal task window shouldn't be in the system modal container, but can happen due to a bug in Android side. (ex. b/111767668) This is a safe guard not to show the frame even if it happens. Bug: 878426 Test: Covered by unittest Change-Id: Ib6344acf0ff9ec86b1e40238a8c8e60828948cee Reviewed-on: https://chromium-review.googlesource.com/1194287 Reviewed-by: Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#588175} [modify] https://crrev.com/5b1c668cf2ddc7418b79a61a7c38a0a3bec86e53/components/exo/client_controlled_shell_surface.cc [modify] https://crrev.com/5b1c668cf2ddc7418b79a61a7c38a0a3bec86e53/components/exo/client_controlled_shell_surface_unittest.cc
,
Sep 6
,
Sep 6
Issue 876488 has been merged into this issue.
,
Sep 7
,
Sep 10
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1859300bbf23c3d2f7d5b91aa5307ecd957670c6 commit 1859300bbf23c3d2f7d5b91aa5307ecd957670c6 Author: Mitsuru Oshima <oshima@chromium.org> Date: Mon Sep 10 14:58:00 2018 Don't allow window frame on shell surfce in system container Normal task window shouldn't be in the system modal container, but can happen due to a bug in Android side. (ex. b/111767668) This is a safe guard not to show the frame even if it happens. Bug: 878426 Test: Covered by unittest Change-Id: Ib6344acf0ff9ec86b1e40238a8c8e60828948cee Reviewed-on: https://chromium-review.googlesource.com/1194287 Reviewed-by: Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#588175}(cherry picked from commit 5b1c668cf2ddc7418b79a61a7c38a0a3bec86e53) Reviewed-on: https://chromium-review.googlesource.com/1216523 Reviewed-by: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/branch-heads/3538@{#215} Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811} [modify] https://crrev.com/1859300bbf23c3d2f7d5b91aa5307ecd957670c6/components/exo/client_controlled_shell_surface.cc [modify] https://crrev.com/1859300bbf23c3d2f7d5b91aa5307ecd957670c6/components/exo/client_controlled_shell_surface_unittest.cc
,
Sep 11
,
Sep 20
,
Sep 26
It's probably too late to merge back to 69, but there are reports that the issue that is fixed in the CL is seen on 69.
,
Oct 2
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by osh...@chromium.org
, Aug 28