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

Issue 819381 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Text selection in isolated iframes cause unwanted page scrolling

Project Member Reported by blois@google.com, Mar 6 2018

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.186 Safari/537.36

Steps to reproduce the problem:
1. Open https://peteblois.github.io/tmp/iframe_scrolling2/
2. Attempt to select the line in the iframe which says 'Try to select this line'.

What is the expected behavior?
The page position remains stable while selecting the text.

What went wrong?
The page position jumps when the text selection begins.

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 64.0.3282.186  Channel: n/a
OS Version: 
Flash Version: 

This appears to rely on site-isolation being enabled.
 
Cc: piatek@google.com
Components: Blink
Labels: Needs-Triage-M64

Comment 3 by tkent@chromium.org, Mar 7 2018

Components: -Blink Internals>Sandbox>SiteIsolation Blink>Editing>Selection
Cc: sindhu.chelamcherla@chromium.org
Components: Blink>Scroll
Labels: -Type-Bug -Pri-2 hasbisect-per-revision ReleaseBlock-Stable Triaged-ET M-65 Target-65 FoundIn-65 OS-Mac OS-Windows Pri-1 Type-Bug-Regression
Owner: skobes@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce this issue on reported version 64.0.3282.186 and latest stable 65.0.3325.146 using Windows 10, Ubuntu 14.04 and Mac 10.13.3. But issue is not seen on latest canary 67.0.3364.0. Hence providing reverse bisect info.

Last Bad Build: 66.0.3344.0
First Good Build: 66.0.3345.0

You are probably looking for a change made after 535635 (known good), but no later than 535637 (first known bad).
CHANGELOG URL:
 https://chromium.googlesource.com/chromium/src/+log/e04a14f4769d6b33b32686ccc3414e917a625ae7..a5676deb1e679035678bc1ed997518cd226378b9

Probably fixed by  https://chromium-review.googlesource.com/857902

@Skobes. Please merge the fix to M-65 if it is safe. Adding RB-Stable for M-65. Please change if not the case.

Thanks!
Cc: pbomm...@chromium.org creis@chromium.org
This is broken on M64 so we're not going to stop M65 further roll out for this. Pls let us know ASAP if there is any concern here. 
Status: WontFix (was: Assigned)
r535636 should not be merged to M65.

Closing based on #4 and #5.

Comment 7 by creis@chromium.org, Mar 9 2018

Labels: -M-65 -Target-65 M-66 Target-66
Status: Fixed (was: WontFix)
More specifically, this is fixed by enabling root layer scrolling in r535636 (66.0.3345.0), correct?  (As opposed to WontFix, that is.)  Updating labels to reflect that it's fixed in M66.

Sign in to add a comment