New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 747801 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:Able to access background page after dragging settings link button from sync overlay.

Reported by vku...@etouch.net, Jul 24 2017

Issue description

Chrome Version:62.0.3165.0 Revision ebd65246aab3b4b1cb5bd462388344195420078d-refs/heads/master@{#488876}(32/64 bit)
OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.11.6,10.12.3)

What steps will reproduce the problem?
(1)Launch chrome and sign to chrome with valid credentials.
(2)Once sync overlay appears drag drop "settings" link button to new tab.
(3)Press 'tab' key and hit 'enter' key once focus reaches 'sync' option, type any data in 'passphrase' field,observe

Actual: Able to access background page after dragging settings link button from sync overlay.

Expected: Should not allow to access background page after dragging settings link button from sync overlay.

This is a regression issue broken in 'M58' and below is the manual regression range
Good Build: 58.0.3020.0 
Bad Build:  58.0.3021.0 

Note: Issue not seen on Mac & Linux OS.


 
ActualResult.mp4
1.1 MB View Download

Comment 1 by vku...@etouch.net, Jul 24 2017

Labels: hasbisect
Owner: msarda@chromium.org
Status: Assigned (was: Unconfirmed)
CL:
https://chromium.googlesource.com/chromium/src/+log/58.0.3020.0..58.0.3021.0?pretty=fuller&n=10000
(Unable to narrow down the range via tool bisect since not able to sign in chromium builds)

Suspecting: 452073 ?
Kindly help to reassign if this is not related to your change.

Comment 2 by msarda@chromium.org, Jul 25 2017

Cc: wittman@chromium.org msw@chromium.org msarda@chromium.org
Components: Internals>Views
Labels: -Pri-1 Pri-2
Owner: msw@chromium.org
CC+ owners of views/constrained_window . Moving this to P2.

The dialog is presented as a modal dialog for the entire browser window - see https://cs.chromium.org/chromium/src/chrome/browser/ui/views/profiles/signin_view_controller_delegate_views.cc?rcl=f88b1b47594ebba08774469a5f236055104c0381&l=128

I expect this is a general problem with the browser modal dialogs. Re-assigning this to msw@ (owner of views/onstrained_window).
msw@: Please re-assign to me if you think that the sign-in code does something wrong when presenting the sync confirmation dialog.


Comment 3 by msw@chromium.org, Sep 21 2017

Cc: sky@chromium.org
Components: UI>Browser>TabStrip UI>Browser
Labels: OS-Chrome OS-Linux
Owner: ----
Status: Available (was: Assigned)
Sorry for the delay. It seems like there are two separate defects:
A) The tab strip accepts drag-and-drop while a modal dialog is open.
B) Focus can be transferred to the underlying window/page while a modal dialog is open.

I can repro (A) on Linux & ChromeOS with the Views-based Edit Bookmark dialog:
(1) Trigger the Bookmark bubble (CTRL+D)
(2) Click "Edit" to open the Edit Bookmark dialog
(3) Drag text from the "Name" or "URL" textfield into the tab strip.
Expected: The modal dialog prevents drag and drop onto the tab strip.
Actual: The tab strip allows drag and drop while a modal window is open.

I can't actually repro (B) on Linux 61.0.3163.79, does it still repro on Windows?
Defect (B) seems even more severe than (A), if it still repros.

Sadly, I'm not actively working in this area, this issue needs a new owner.
Project Member

Comment 4 by sheriffbot@chromium.org, Sep 24

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Components: -UI>Settings

Sign in to add a comment