New issue
Advanced search Search tips

Issue 596780 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Google.com is flickery with spv2

Project Member Reported by pdr@chromium.org, Mar 22 2016

Issue description

Visiting most non-trivial websites is a flickery experience with slimming paint v2 enabled. This is due to how subsequence caching works with spv2.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 22 2016

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

commit bfb0fc4dbd0648aac537ca64ad6fffad621eb6be
Author: pdr <pdr@chromium.org>
Date: Tue Mar 22 20:55:04 2016

Disable subsequence caching for spv2

Invalidation does not currently work properly in spv2 and subsequence
caching would cause subtrees to incorrectly paint when spv2 was enabled.
On real websites this results in flashing content that should have
painted but didn't, or flashes of cached content incorrectly drawn at
the origin.

This patch disables subsequence caching which fixes google.com and
several layout tests.

BUG= 596780 

Review URL: https://codereview.chromium.org/1820793004

Cr-Commit-Position: refs/heads/master@{#382675}

[modify] https://crrev.com/bfb0fc4dbd0648aac537ca64ad6fffad621eb6be/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/bfb0fc4dbd0648aac537ca64ad6fffad621eb6be/third_party/WebKit/Source/platform/graphics/paint/SubsequenceRecorder.cpp

Comment 2 by pdr@chromium.org, Mar 22 2016

Status: Fixed (was: Started)

Sign in to add a comment