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

Issue 818519 link

Starred by 6 users

Issue metadata

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



Sign in to add a comment

some embedded elements in twitter make next twits to be hidden or flickering

Reported by vsemozhe...@gmail.com, Mar 4 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3361.0 Safari/537.36

Example URL:
Any twitter timeline

Steps to reproduce the problem:
1. Open https://twitter.com/googlechrome
2. Slowly scroll down to the twit as of 4 Oct 2017 (embedded post) or  6 Jun 2017 (embedded video) and a bit below them.

What is the expected behavior?
All next twits are visible and stable.

What went wrong?
Some ext twits (1-2) are hidden with big blank space or flickering, till more scrolling down.

Does it occur on multiple sites: N/A

Is it a problem with a plugin? N/A 

Did this work before? Yes At least 66.0.3355.0 dev (64-bit) is OK now.

Does this work in other browsers? Yes

Chrome version: 67.0.3361.0  Channel: canary
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: 29.0.0.113
 
Not sure if this is related:

https://hackernoon.com/please-stop-using-console-log-its-broken-b5d7d396cf15

In 66.0.3355.0 dev all is OK
In 67.0.3361.0 canary all comments and embedded post in the bottom part of the page are hidden.

Comment 2 by woxxom@gmail.com, Mar 4 2018

Bisect info: 540677 (good) - 540690 (bad)
https://chromium.googlesource.com/chromium/src/+log/359b23a6..8187e24c?pretty=fuller
Suspecting r540687 = 8e4234fdc272ddda5ec9c156a2397c64d7a69f60 = https://crrev.com/c/946560 by chrishtr@chromium.org
"[RLS] Don't include composited scroll offset in squashing offset from transformed ancestor."
Landed in 67.0.3360.0

Tangentially related: scrolling the page makes all video images flicker their black placeholder background, which looks really bad in all versions of Chrome. There's no such flickering in Firefox.
Labels: Needs-Bisect Needs-Triage-M67
Cc: susan.boorgula@chromium.org
Components: Blink>Scroll
Labels: -Pri-2 -Type-Compat -Needs-Bisect hasbisect-per-revision ReleaseBlock-Stable Triaged-ET RegressedIn-67 M-67 Target-67 FoundIn-67 OS-Linux OS-Mac Pri-1 Type-Bug-Regression
Owner: chrishtr@chromium.org
Status: Assigned (was: Unconfirmed)
vsemozhetbyt@ Thanks for the issue.

Tested this issue on Mac OS 10.12.6, Windows 10 and Ubuntu 14.04 on the latest Canary 67.0.3361.0 and able to reproduce the issue by following the steps mentioned above.

Bisect Information:
===================
Good Build: 66.0.3359.0 (Revision - 540276)
Bad Build : 67.0.3360.0 (Revision - 540723)

On executing the per-revision bisect script, below is the Changelog URL:

https://chromium.googlesource.com/chromium/src/+log/c7056ddb3e81ad2e9a1fdd8fe48c6b65d400153b..8e4234fdc272ddda5ec9c156a2397c64d7a69f60

From the above Changelog, suspecting the below change:
Reviewed-on:  https://chromium-review.googlesource.com/946560

chrishtr@ Please check and confirm if this issue is related to your change, else help us in assigning to the right owner.

Adding ReleaseBlock-Stable as this is a recent regression. Please feel free to remove it if it is not applicable.

Thanks.
Status: Started (was: Assigned)
Cc: skobes@chromium.org chrishtr@chromium.org szager@chromium.org
 Issue 818464  has been merged into this issue.
Project Member

Comment 7 by bugdroid1@chromium.org, Mar 6 2018

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

commit d020616eac00b09e1da3dbd36bbcce28502f72b7
Author: Chris Harrelson <chrishtr@chromium.org>
Date: Tue Mar 06 00:45:17 2018

[RLS] Add in scrolling to squashed layers when computing interest rect

This was the case until
https://chromium.googlesource.com/chromium/src/+/8e4234fdc272ddda5ec9c156a2397c64d7a69f60,
which removed scroll offset.

Bug:818519

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Id79668c0ea9ca62ff8c181de2b4f1a0be6594e82
Reviewed-on: https://chromium-review.googlesource.com/949969
Reviewed-by: vmpstr <vmpstr@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541007}
[modify] https://crrev.com/d020616eac00b09e1da3dbd36bbcce28502f72b7/third_party/WebKit/Source/core/paint/compositing/CompositedLayerMapping.cpp
[modify] https://crrev.com/d020616eac00b09e1da3dbd36bbcce28502f72b7/third_party/WebKit/Source/core/paint/compositing/CompositedLayerMappingTest.cpp

Status: Fixed (was: Started)
Labels: OS-Android
67.0.3364.0 seems OK, thank you!
Cc: schenney@chromium.org
 Issue 819480  has been merged into this issue.
Cc: rpop@chromium.org
 Issue 819117  has been merged into this issue.

Sign in to add a comment