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

Issue 682655 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 677794
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:Page appears to be autoscrolled after pressing Esc key in "www.toolsqa.com"

Reported by adha...@etouch.net, Jan 19 2017

Issue description

Chrome Version: 57.0.2986.0 (Official Build) 12c8d70173fc1a04fa588b75be71ab5e80655c60-refs/heads/master@{#444600} (32/64-bit)
OS: Windows (7,8,10), Mac (10.11.6, 10.12.1), Linux (14.04 LTS)

Test url:http://toolsqa.com/selenium-webdriver/selenium-webdriver-basics-video-tutorials/

What steps will reproduce the problem?
(1)Launch chrome and navigate to the above url.
(2)Play the first video under 'Selenium Browser Command Video'
(3)Click on full screen icon and press Esc.(Kindly refer the video)
(4)Observe.

Actual:Page appears to be autoscrolled after pressing Esc key.

Expected:Page should not auto scroll after pressing Esc key.

This is a Regression issue broken in M-56,will soon update other info.

Good build:56.0.2895.0
Bad build:56.0.2896.0
 
Actual result.mp4
542 KB View Download
Expected result.mp4
767 KB View Download

Comment 1 by bokan@chromium.org, Jan 19 2017

Labels: Hotlist-Input-Dev Needs-Bisect
Owner: bokan@chromium.org
Status: Available (was: Unconfirmed)
Seems similar to  issue 649655  which was caused by scroll anchoring. Could we get a per-revision bisect result?
Cc: rbasuvula@chromium.org bokan@chromium.org
Labels: -Needs-Bisect hasbisect-per-revision
Owner: skobes@chromium.org
Status: Assigned (was: Available)
Using the per-revision bisect providing the bisect results,
Good build: 56.0.2895.0 (Revision: 426105).
Bad build: 56.0.2896.0 (Revision: 426358).

You are probably looking for a change made after 425789 (known good), but no later than 425790 (first known bad).

CHANGE-LOG URL:
---------------
https://chromium.googlesource.com/chromium/src/+log/bf2f3855e7e22042ad5d11e785b13d6e7c84127a..6d380ba3ab246a15d6269a75328f481336b53ee1


From the CL above, assigning the issue to the concern owner

@ skobes : 
------------------
Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Review-Url: https://chromiumcodereview.appspot.com/2421263003

Note : Able to reproduce the issue in Ubuntu 14.04, win 10.0 & Mac 10.12.2

Comment 3 by skobes@chromium.org, Jan 21 2017

Labels: -Pri-1 Pri-2
Likely related to  issue 677794 .

Comment 4 by skobes@chromium.org, Jan 25 2017

Mergedinto: 677794
Status: Duplicate (was: Assigned)
This is  issue 677794  interacting with our fullscreen scrolling behavior.

FullscreenController scrolls the main frame to 0 when the video goes fullscreen, and restores the original scroll position when we exit fullscreen mode.

The website reacts by switching the sticky header off and on again, causing content shifts (see  http://crbug.com/677794#c10 ).

ScrollAnchor performs an adjustment when exiting fullscreen mode, causing the observed behavior.  This adjustment occurs after FullscreenController's scroll restoration, because the scroll event that causes the content shift fires asynchronously.

ScrollAnchor does not perform the inverse adjustment when entering fullscreen mode, because the scroll offset is 0 at that time.

I'm duping this into  issue 677794 , but I've also filed  issue 684920  to investigate why FullscreenController has to scroll the page.

Sign in to add a comment