New issue
Advanced search Search tips

Issue 850335 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 843001



Sign in to add a comment

App on launcher clicked should open in that display rather than one in focus

Project Member Reported by timzheng@chromium.org, Jun 7 2018

Issue description

Google Chrome	68.0.3440.15 (Official Build) dev (64-bit)
Revision	97fda87c8faca0ed36579987237d552dd92d1fcd-refs/branch-heads/3440@{#179}
Platform	10718.13.0 (Official Build) dev-channel caroline-ndktranslation
Firmware Version	Google_Caroline.7820.356.0
Customization ID	SAMSUNG-CAROLINE

Steps to reproduce the problem:
1. Connect Chromebook to a external display in extended mode
2. Click on a tab in Chromebook screen to put it in focus
3. Click on gmail app from the extended screen launcher

What is the expected behavior?
The gamil app opens as a window display that contains the launcher clicked.

What went wrong?
The gmail app will open as a tab in the display that had the tab in focus.

This is a related bug that tracked the fix of the shelf click. Now it's inconsistent between the shelf click and launcher click.
https://bugs.chromium.org/p/chromium/issues/detail?id=586091
 
Blocking: 843001
Owner: weidongg@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 7 2018

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

commit df2bd9a61bad5d6297454ae1da7b00d6b4df5792
Author: Weidong Guo <weidongg@chromium.org>
Date: Thu Jun 07 21:59:47 2018

Launch app from the display where it is clicked

Changes:
We fixed this for shelf before
(https://codereview.chromium.org/2894743002), but not for
app launcher which results in inconsistent behavior. So
fix this for launcher as well.

Bug: 850335
Change-Id: I19849658ad0617fe7326895b58efd72261eb8a8c
Reviewed-on: https://chromium-review.googlesource.com/1091277
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565429}
[modify] https://crrev.com/df2bd9a61bad5d6297454ae1da7b00d6b4df5792/chrome/browser/ui/app_list/app_list_controller_impl.cc

Sign in to add a comment