App Launcher not immediately dismissed when selecting Arc and Crostini apps |
||
Issue descriptionThe logic to immediately dismiss the app launcher when launching Arc and Crostini apps no longer works (i.e. the DismissView() calls in CrostiniAppItem::Activate(), ArcAppResult::Open(), ..). I manually confirmed that this was a result of the change https://chromium-review.googlesource.com/c/chromium/src/+/1056447. hejq@, can you please take a look?
,
May 16 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/30cde66f48c556315f1763f4fc75d6a0bccd64b3 commit 30cde66f48c556315f1763f4fc75d6a0bccd64b3 Author: Jiaquan He <hejq@google.com> Date: Wed May 16 16:33:12 2018 app_list: fix app opening bug. Regression of crrev.com/c/1056447. Bug: 733662 Bug: 843451 Change-Id: Ia123a20a0c6fff9ddb20697e10e8423b3f04b949 Reviewed-on: https://chromium-review.googlesource.com/1061835 Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Jiaquan He <hejq@google.com> Cr-Commit-Position: refs/heads/master@{#559133} [modify] https://crrev.com/30cde66f48c556315f1763f4fc75d6a0bccd64b3/chrome/browser/ui/app_list/app_list_client_impl.cc
,
May 30 2018
|
||
►
Sign in to add a comment |
||
Comment 1 by hejq@chromium.org
, May 16 2018Status: Started (was: Assigned)